Skip to content

Set default PR reviewer - #28

Merged
hayarobi merged 1 commit into
mainfrom
topic/automatic-pr-reviewer
Aug 11, 2026
Merged

Set default PR reviewer#28
hayarobi merged 1 commit into
mainfrom
topic/automatic-pr-reviewer

Conversation

@hayarobi

Copy link
Copy Markdown
Member

This PR sets the default reviewer for PRs.

@hayarobi
hayarobi requested a review from ultrafellen August 11, 2026 07:00
@hayarobi
hayarobi marked this pull request as draft August 11, 2026 07:20
@hayarobi

Copy link
Copy Markdown
Member Author

There are some security issue(s) on this PR. I will fix it them and merge PR.

@hayarobi
hayarobi force-pushed the topic/automatic-pr-reviewer branch from 1c25bff to 2791b39 Compare August 11, 2026 10:31
@hayarobi hayarobi self-assigned this Aug 11, 2026
@hayarobi

Copy link
Copy Markdown
Member Author

It appears that image-size, which docusaurus depends on, has a security issue, and there is no fix available yet.

This problem persists even after updating docusaurus to the latest version, so I will update it immediately once a version with this issue resolved is released.

@ultrafellen

ultrafellen commented Aug 11, 2026

Copy link
Copy Markdown
Member

The failing build check is unrelated to this PR. The build itself succeeds — it is the following npm audit --audit-level=high step that fails, due to an unpatched CVE in image-size (GHSA-w3rx-r6r6-pgpr) pulled in via @docusaurus/mdx-loader. Every PR opened right now fails the same way. It is not a required status check, so merging is unaffected — leaving it red until we address it in a separate PR.

This PR is approved and mergeable as is, so go ahead and merge it.

@ultrafellen

Copy link
Copy Markdown
Member

Follow-up: #29 fixes this at the source — it raises the overrides for the two fixable advisories and allowlists the two unpatchable image-size ones. Its own build check is green.

Note that merging #29 will not turn this PR green on its own: a pull_request workflow is not re-triggered by base branch updates. Rebase this branch after #29 lands and the check will pass — or merge as-is, since the check is not required either way.

@hayarobi
hayarobi marked this pull request as ready for review August 11, 2026 15:20
@hayarobi
hayarobi merged commit 72564b3 into main Aug 11, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants