-
Notifications
You must be signed in to change notification settings - Fork 649
Limit the branches that travis builds on #1991
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
Conversation
Because branches that become PRs are built twice :(
r? @smarnach (rust_highfive has picked a reviewer for you, use r? to override) |
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.
Looks good to me.
branches: | ||
only: | ||
- auto | ||
- try |
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.
What is this branch? We don't currently seem to have a branch with this name.
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.
It's used when you do @bors try
.
📌 Commit fcee1b2 has been approved by |
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 saw it mentioned somewhere that we don't need to run tests on @bors r+ |
💡 This pull request was already approved, no need to approve it again. |
📌 Commit fcee1b2 has been approved by |
@bors retry |
Have we reached some quota limit for Travis builds? This didn't even start building on Travis yet, same for #1929. |
hmm, or @bors can run X number of jobs in parallel and the jobs that "timed out" are still running and blocking new jobs from being accepted 🤔 |
Limit the branches that travis builds on Because branches that become PRs are built twice :(
☀️ Test successful - checks-travis |
Because branches that become PRs are built twice :(