Skip to content

Add pr_incremental option#88

Merged
ewokcami merged 17 commits into
mainfrom
pr-incremental
Sep 18, 2025
Merged

Add pr_incremental option#88
ewokcami merged 17 commits into
mainfrom
pr-incremental

Conversation

@ewokcami
Copy link
Copy Markdown
Contributor

@ewokcami ewokcami commented Sep 5, 2025

Allow users to speed up PR scans with the --pr-incremental option when using our github-action.

Comment thread README.md
Comment thread README.md Outdated
Comment thread src/scan.ts Outdated
Comment thread dist/index.js
core.error("The GitHub PR ID must be available for `pr_incremental` to work");
}
else {
options.push(`--pr-incremental=true`);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to add another option? This is already set, right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only place it is set.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My mistake, thanks!

Comment thread dist/index.js Outdated
@ewokcami ewokcami requested a review from hghmn September 10, 2025 06:30
dkourkouzelis
dkourkouzelis previously approved these changes Sep 10, 2025
hghmn
hghmn previously approved these changes Sep 10, 2025
IAmATeaPot418
IAmATeaPot418 previously approved these changes Sep 18, 2025
@ewokcami ewokcami dismissed stale reviews from IAmATeaPot418 and hghmn via 365a360 September 18, 2025 18:35
@ewokcami ewokcami requested a review from hghmn September 18, 2025 20:29
@ewokcami ewokcami merged commit fc47e4d into main Sep 18, 2025
3 checks passed
@ewokcami ewokcami deleted the pr-incremental branch September 18, 2025 20:47
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.

4 participants