Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: limit continuous fuzzing to dev branch #1101

Merged
merged 1 commit into from
Feb 14, 2025
Merged

refactor: limit continuous fuzzing to dev branch #1101

merged 1 commit into from
Feb 14, 2025

Conversation

bowenli86
Copy link
Member

@bowenli86 bowenli86 commented Feb 14, 2025

Motivation:

per chat w/ @0xClandestine , given continuous fuzzing tests takes too long (currently ~2.5h ish), we wanna make it only run when a commit is merged to dev branch, instead of every PR change

it should make the CI passes as a whole a lot faster, and save us huge money spent on github actions

Modifications:

make continuous fuzzing only run when a commit is merged to dev branch, instead of every PR change

Result:

After your change, what will change.

@bowenli86 bowenli86 marked this pull request as ready for review February 14, 2025 05:39
@bowenli86
Copy link
Member Author

image

you can see as part of this PR that continuous fuzzing has been skipped

@0xClandestine 0xClandestine changed the title test: make continuous fuzzing tests runs only when commit is pushed to dev branch refactor: limit continuous fuzzing to dev branch Feb 14, 2025
Copy link
Member

@0xClandestine 0xClandestine left a comment

Choose a reason for hiding this comment

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

lgtm

@0xClandestine 0xClandestine added the enhancement New feature or request label Feb 14, 2025
@bowenli86 bowenli86 merged commit e203f3b into dev Feb 14, 2025
9 checks passed
@bowenli86 bowenli86 deleted the feat/fuzz branch February 14, 2025 16:46
ypatil12 pushed a commit that referenced this pull request Feb 19, 2025
**Motivation:**

given continuous fuzzing tests takes too
long (currently ~2.5h ish), we wanna make it only run when a commit is
merged to dev branch, instead of every PR change

it should make the CI passes as a whole a lot faster, and save us huge
money spent on github actions

**Modifications:**

make continuous fuzzing only run when a commit is merged to dev branch,
instead of every PR change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants