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
Align travis build to actions #1048
Conversation
Failure that occurs only on nightly seems to be more often due to Rust breakage than a bug. Take deprecation warnings more serious by actively denying them which warns us of incompatible future versions as nightly would. Note: May remove this again in the future if it doesn't help. We've already had the case where the rustc version commitment left us without a proper replacement for a while since that had only been introduce shortly before deprecation. Also ensures that tests are ran on all relevant builds instead of only on nightly.
For osx and windows targets build with the first env combination since they are matrix includes. This ensures that this selects the build script with default features instead of no features.
3220ece
to
f0bef52
Compare
Does it resolve #1043? |
Yes it should. Thank you for noting. |
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
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.
Reviewed the executed Travis jobs and found osx
suspiciously fast. Why is scripting so hard :/
d64eeb3
to
f4bb180
Compare
Checks if a variable was declared, not if it has a non-empty content. The core operation `${VARNAME+word}` will equal `word` if `VARNAME` is set and non-null and be empty otherwise. ``` DEFAULT_FEATURES='' [ -z "$DEFAULT_FEATURES" ] # true ``` ``` DEFAULT_FEATURES='' [ -n "${DEFAULT_FEATURES+exists}" ] # false ```
f4bb180
to
5a03954
Compare
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 better, it's smarter than I came up with!
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
Failure that occurs only on nightly seems to be more often due to Rust
breakage than a bug. Take deprecation warnings more serious by actively
denying them which warns us of incompatible future versions as nightly
would.
Note: May remove this again in the future if it doesn't help. We've
already had the case where the rustc version commitment left us
without a proper replacement for a while since that had only been
introduce shortly before deprecation.
Also ensures that tests are ran on all relevant builds instead of only
on nightly.
Closes: #1043