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

Parameterize nightly toolchain in CI #5329

Closed
wants to merge 8 commits into from

Conversation

cart
Copy link
Member

@cart cart commented Jul 15, 2022

Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

  • cargo-udeps requires nightly and likely always will.
  • Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
  • Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

Solution

This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.

@cart cart added the A-Build-System Related to build systems or continuous integration label Jul 15, 2022
@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

bors bot added a commit that referenced this pull request Jul 15, 2022
@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Already running a review

@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Already running a review

@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Build failed:

@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

bors bot added a commit that referenced this pull request Jul 15, 2022
@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Already running a review

@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Build failed:

@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

bors bot added a commit that referenced this pull request Jul 15, 2022
@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Already running a review

@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Build failed:

@cart
Copy link
Member Author

cart commented Jul 15, 2022

bors try

bors bot added a commit that referenced this pull request Jul 15, 2022
@cart
Copy link
Member Author

cart commented Jul 15, 2022

closing in favor of #5330. This impl doesn't work due to Github Actions limitations.

@cart cart closed this Jul 15, 2022
bors bot pushed a commit that referenced this pull request Jul 15, 2022
# Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

* cargo-udeps requires nightly and likely always will.
* Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
* Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

## Solution

* This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.
* Put nightly jobs to check wasm and nightly on their own matrix. I also removed tests on nightly linux, just build check
* alternative to #5329
bors bot pushed a commit that referenced this pull request Jul 15, 2022
# Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

* cargo-udeps requires nightly and likely always will.
* Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
* Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

## Solution

* This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.
* Put nightly jobs to check wasm and nightly on their own matrix. I also removed tests on nightly linux, just build check
* alternative to #5329
bors bot pushed a commit that referenced this pull request Jul 15, 2022
# Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

* cargo-udeps requires nightly and likely always will.
* Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
* Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

## Solution

* This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.
* Put nightly jobs to check wasm and nightly on their own matrix. I also removed tests on nightly linux, just build check
* alternative to #5329
@bors
Copy link
Contributor

bors bot commented Jul 15, 2022

try

Timed out.

bors bot pushed a commit that referenced this pull request Jul 15, 2022
# Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

* cargo-udeps requires nightly and likely always will.
* Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
* Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

## Solution

* This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.
* Put nightly jobs to check wasm and nightly on their own matrix. I also removed tests on nightly linux, just build check
* alternative to #5329

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
inodentry pushed a commit to IyesGames/bevy that referenced this pull request Aug 8, 2022
# Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

* cargo-udeps requires nightly and likely always will.
* Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
* Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

## Solution

* This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.
* Put nightly jobs to check wasm and nightly on their own matrix. I also removed tests on nightly linux, just build check
* alternative to bevyengine#5329

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
james7132 pushed a commit to james7132/bevy that referenced this pull request Oct 28, 2022
# Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

* cargo-udeps requires nightly and likely always will.
* Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
* Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

## Solution

* This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.
* Put nightly jobs to check wasm and nightly on their own matrix. I also removed tests on nightly linux, just build check
* alternative to bevyengine#5329

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

Rust's nightly builds semi-regularly break us (or our dependencies). This creates churn and angst when we're just trying to get our jobs done.

We do still want nightly builds for a variety of reasons:

* cargo-udeps requires nightly and likely always will.
* Helps us catch rust nightly bugs quickly. We're "good citizens" if we regularly report regressions.
* Lets us prepare for "actual expected breakage" ahead of stable releases so we avoid breaking main users.

## Solution

* This pr parameterizes the nightly toolchain, making it an easy one-liner to pin our builds to a specific nightly, when required.
* Put nightly jobs to check wasm and nightly on their own matrix. I also removed tests on nightly linux, just build check
* alternative to bevyengine#5329

Co-authored-by: Carter Anderson <mcanders1@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Build-System Related to build systems or continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants