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

Add various options #42

Merged
merged 8 commits into from
Jul 22, 2019
Merged

Add various options #42

merged 8 commits into from
Jul 22, 2019

Conversation

jonhoo
Copy link
Collaborator

@jonhoo jonhoo commented Jul 21, 2019

This PR adds support for a few requested features (#37 and #41), as well as for running tests single-threaded (--test-threads=1), which some projects need.

Note that errors on nightly are ignored, because nightly is
unpredictable.

Fixes #37
@jonhoo jonhoo requested a review from epage July 21, 2019 17:21
@jonhoo
Copy link
Collaborator Author

jonhoo commented Jul 21, 2019

cc @djc

@djc
Copy link
Contributor

djc commented Jul 21, 2019

Awesome!

azure/test.yml Show resolved Hide resolved
@codecov-io
Copy link

codecov-io commented Jul 22, 2019

Codecov Report

Merging #42 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #42   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines           5      5           
=====================================
  Hits            5      5
Impacted Files Coverage Δ
src/lib.rs 100% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dbd8115...29e081a. Read the comment docs.

@jonhoo
Copy link
Collaborator Author

jonhoo commented Jul 22, 2019

Pushed one more option that enables you to skip cargo check --all-features, since some projects have features that are nightly only, or otherwise should not generally be enabled.

jonhoo added a commit to tokio-rs/tracing that referenced this pull request Jul 22, 2019
We need support for _not_ running with `--all-features` due to the
nightly requirement for async/await which is enabled by a feature in
`tracing`. This will eventually land with
crate-ci/azure-pipelines#42.
hawkw pushed a commit to tokio-rs/tracing that referenced this pull request Jul 22, 2019
* First draft of Azure Pipelines CI config

You'll have to follow the instructions over at
https://github.com/crate-ci/azure-pipelines to get it fully set up.
Might also run into some weirdness with having to change the "default"
branch once merged. But 🎉

Oh, yeah, and need to also update Cargo.toml, README badge, etc.

* Also check benchmarks

* Pool is set per job, not per stage

* Not a template anymore

* Use correct service connection name

* Disable readme doctest pending rewrite

* Fully remove old doctest readme

* Don't run cargo check yet

We need support for _not_ running with `--all-features` due to the
nightly requirement for async/await which is enabled by a feature in
`tracing`. This will eventually land with
crate-ci/azure-pipelines#42.

* Hack dependency chain

Closes #177
@jonhoo jonhoo merged commit 8f5acef into master Jul 22, 2019
@jonhoo jonhoo deleted the testing-params branch July 22, 2019 19:53
hawkw pushed a commit to tokio-rs/tracing that referenced this pull request Jul 22, 2019
Brings back `cargo check` as discussed here: 
#191 (comment) 
now that crate-ci/azure-pipelines#42 has landed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants