Skip to content

Commit

Permalink
Changes 2.0
Browse files Browse the repository at this point in the history
- Add an axum-extractor
- Improve some code comments
- Have Api `Err` responses return `(StatusCode, axum::Json<Value>)`
- Move Utility code for routes
- Remove the `json_error()` response-builder, as it wasn't that usefull
- Add `clippy::unwrap_used` to clippy denies
  • Loading branch information
TristanJSchoenmakers committed Dec 28, 2023
1 parent 7e0504b commit 92af177
Show file tree
Hide file tree
Showing 15 changed files with 747 additions and 392 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,5 +39,5 @@ jobs:
- run: sqlx migrate run
- run: cargo build --bins --tests --release
- run: cargo test --release
- run: cargo clippy --all-targets --all-features -- -D warnings
- run: cargo clippy --all-targets --all-features -- -D warnings -D clippy::unwrap_used
- run: cargo fmt --all -- --check
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,14 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [0.2.0] - 2023-12-28
- Add an [axum-extractor](https://docs.rs/axum/latest/axum/#extractors) for extracting and validating Json in a request
- Improve some code comments
- Have Api `Err` responses return `(StatusCode, axum::Json<Value>)` instead of `(StatusCode, String)`
- Move Utility code for routes to *extractors.rs* and *response_builders.rs*
- Remove the `json_error()` response-builder, as it wasn't that usefull
- Add `clippy::unwrap_used` to clippy denies

## [0.1.3] - 2023-12-07
- Removed dev-dependency `hyper`, using `axum::http` instead
- Removed `cargo audit`, has not proved to be usefull
Expand Down
Loading

0 comments on commit 92af177

Please sign in to comment.