-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Test PR #1794
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
Test PR #1794
Conversation
Added GitHub checks aliases for unit tests.
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.
Pull Request Overview
This PR adds GitHub checks aliases to the Evergreen configuration to enable visibility of Evergreen build status directly in GitHub commits.
- Added
github_checks_aliasesconfiguration section - Configured aliases for unit test variants to display in GitHub
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| ls -la | ||
| df -h | ||
| # Allows to see evergreen checks in GitHub commits |
Copilot
AI
Oct 16, 2025
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.
Grammar correction: 'Allows to see' should be 'Allows viewing' or 'Allows you to see'.
| # Allows to see evergreen checks in GitHub commits | |
| # Allows you to see evergreen checks in GitHub commits |
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.
I disagree 😁
| github_checks_aliases: | ||
| - variant: "unit-tests-.*" | ||
| task: ".*" | ||
|
|
Copilot
AI
Oct 16, 2025
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.
Remove trailing whitespace on line 31.
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.
I disagree as well
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.
Overall it looks good. I suppose we're going to do the same for all tests once we get rid of our flaky tests?
Added GitHub checks aliases for unit tests.