-
Notifications
You must be signed in to change notification settings - Fork 508
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
🌱 Add Dart and Flutter CI systems to CI tests check. #1548
Conversation
Integration tests success for |
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.
Thank you!
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.
Thanks!
Integration tests success for |
Fixed the gofmt issue for the PR #1548
The current check is looking at the github checks data to identify whether a given PR ran tests. Flutter and Dart repos are failing the check becuase their systems are not recognized as CI Systems. Bug: ossf#1547
20c055b
to
145422b
Compare
Integration tests success for |
The current check is looking at the github checks data to identify whether a given PR ran tests. Flutter and Dart repos are failing the check because their systems are not recognized as CI Systems.
Bug: #1547
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Updates the CI Test check adding new CI Systems.
What is the current behavior? (You can also link to an open issue here)
Only common CI systems are included in the CI Test check and the ones used by Dart and Flutter are not included.
#1547
What is the new behavior (if this is a feature change)?
flutter-dashboard and Cirrus CI will now be recognized as CI systems running tests on the repositories' PRs
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No
Other information: