Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: Make testfast actually fast #273

Merged
merged 1 commit into from
Sep 30, 2020
Merged

ci: Make testfast actually fast #273

merged 1 commit into from
Sep 30, 2020

Conversation

Swatinem
Copy link
Member

turns out, changing RUSTFLAGS invalidates all the caches, thats why the checkfast/fastfast job was not using it properly.

@Swatinem
Copy link
Member Author

lol, all the other tests are broken by http-rs/surf#244 right now. So yes, we will definitely change our Cargo.toml soon, lol.

@Swatinem Swatinem merged commit cd549de into master Sep 30, 2020
@Swatinem Swatinem deleted the ci/fast-fast branch September 30, 2020 12:40
jan-auer added a commit that referenced this pull request Nov 25, 2020
* master: (59 commits)
  fix: Correctly apply environment from env (#293)
  fix: Make Rust 1.48 clippy happy (#294)
  docs: Document integrations and the Hub better (#291)
  ref: Remove deprecated error-chain and failure crates (#290)
  release: 0.21.0
  meta: Update Changelog
  feat: End sessions with explicit status (#289)
  fix: Scope transaction name can be overriden in sentry-actix (#287)
  fix: sentry-actix should not capture client errors (#286)
  fix: Clean up sentry-actix toml (#285)
  ref: Remove empty integrations (#283)
  feat: Add support for actix-web 3 (#282)
  feat: Preliminary work to integrate Performance Monitoring (#276)
  ref: Introduce a SessionFlusher (#279)
  fix: Set a default environment based on debug_assertions (#280)
  ref: Rearchitect the log and slog Integrations (#268)
  ref: Deprecate public fields on Integrations (#267)
  ci: Make testfast actually fast (#273)
  fix: Update surf and unbreak CI (#274)
  ci: Use smarter cache action (#272)
  ...
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants