Skip to content

chore: Update to rust 2024 edition + rustfmt 2024 style edition - #662

Merged
spencewenski merged 1 commit into
mainfrom
rust-2024
Feb 26, 2025
Merged

spencewenski merged 1 commit into
mainfrom
rust-2024

Conversation

@spencewenski

Copy link
Copy Markdown
Member

This mostly just required a bunch of style changes (automatically applied wiht cargo fmt)

This mostly just required a bunch of style changes (automatically
applied wiht `cargo fmt`)
@codecov

codecov Bot commented Feb 26, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 13.04348% with 140 lines in your changes missing coverage. Please review.

Project coverage is 60.27%. Comparing base (8b9aae3) to head (f597523).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/app/roadster_app.rs 9.92% 127 Missing ⚠️
src/app/mod.rs 0.00% 6 Missing ⚠️
src/app/context.rs 75.00% 2 Missing ⚠️
src/lifecycle/db/sea_orm/graceful_shutdown.rs 0.00% 2 Missing ⚠️
src/service/worker/sidekiq/service.rs 0.00% 2 Missing ⚠️
src/service/http/builder.rs 50.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/api/cli/mod.rs 95.78% <ø> (ø)
src/api/cli/roadster/health.rs 0.00% <ø> (ø)
src/api/cli/roadster/list_routes.rs 0.00% <ø> (ø)
src/api/cli/roadster/migrate.rs 0.00% <ø> (ø)
src/api/cli/roadster/mod.rs 61.53% <ø> (ø)
src/api/cli/roadster/open_api.rs 0.00% <ø> (ø)
src/api/cli/roadster/print_config.rs 57.89% <ø> (ø)
src/api/http/health.rs 79.77% <ø> (ø)
src/api/http/ping.rs 73.07% <ø> (ø)
src/app/prepare.rs 82.87% <ø> (ø)
... and 55 more

... and 3 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@spencewenski
spencewenski merged commit b626737 into main Feb 26, 2025
@spencewenski
spencewenski deleted the rust-2024 branch February 26, 2025 07:27
spencewenski added a commit that referenced this pull request Mar 5, 2025
## 🤖 New release

* `roadster`: 0.7.0-beta -> 0.7.0-beta.1 (⚠ API breaking changes)

### ⚠ `roadster` breaking changes

```text
--- failure auto_trait_impl_removed: auto trait no longer implemented ---

Description:
A public type has stopped implementing one or more auto traits. This can break downstream code that depends on the traits being implemented.
        ref: https://doc.rust-lang.org/reference/special-types-and-traits.html#auto-traits
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.39.0/src/lints/auto_trait_impl_removed.ron

Failed in:
  type PrepareOptions is no longer UnwindSafe, in /tmp/.tmpU0JW3g/roadster/src/app/prepare.rs:76
  type PrepareOptions is no longer RefUnwindSafe, in /tmp/.tmpU0JW3g/roadster/src/app/prepare.rs:76
```

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.7.0-beta.1](roadster-v0.7.0-beta...roadster-v0.7.0-beta.1)
- 2025-03-04

### Added

- Allow logging sensitive headers in the dev environment
([#666](#666))
- Allow overriding config fields or entire config
([#661](#661))

### Fixed

- Export `TestAppState` to allow for external use
([#672](#672))

### Other

- Add non-async `Source` to config chapter in book
([#670](#670))
- Update testing chapter in book
([#668](#668))
- Update book deploy workflow to only run if book tests pass on main
([#665](#665))
- Update all workspace members to rust 2024 edition
([#664](#664))
- Improve diesel docs in book
([#663](#663))
- Update to rust 2024 edition + rustfmt 2024 style edition
([#662](#662))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Spencer Ferris <3319370+spencewenski@users.noreply.github.com>
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.

1 participant