-
Notifications
You must be signed in to change notification settings - Fork 109
Add proper labeling and java run stage #777
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
base: master
Are you sure you want to change the base?
Conversation
fbf5b2d to
b8c00e0
Compare
|
|
||
| jobs: | ||
| get-all-metadata: | ||
| if: github.repository == 'oracle/graalvm-reachability-metadata' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to run this job on master push on forks (but not the test-all-metadata and all-metadata-passed)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This one is not automatic, just manual. Although I have a follow up PR:
https://github.com/oracle/graalvm-reachability-metadata/pulls
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But it should still trigger on pushes to master (even on a fork), no? The test-all-metadata and all-metadata-passed jobs are still behind if: github.repository == 'oracle/graalvm-reachability-metadata' checks, but the get-all-metadata job isn't. Is this by design?
|
|
||
| jobs: | ||
| get-changed-metadata: | ||
| if: github.repository == 'oracle/graalvm-reachability-metadata' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same question as for test-all-metadata.yml.
What does this PR do?
Does smarter bisecting when running automated tests. Check if it works first, and only if it fails do the bisecting. Also, adds a new category
java runand applies labels.Also, increases the frequency of running the testing job to daily.
Fixes: #760