more ci updates#32
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces updates to the CI configuration by refining justfile recipes and enhancing documentation tasks, along with adding a new Codecov badge to the README.
- Updated the "check-if-published" recipe to incorporate a package parameter and adjust version retrieval.
- Modified the "docs" and "test-doc" recipes to improve documentation generation.
- Added a Codecov badge to the README for better code coverage visibility.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| justfile | Updated recipes for check, docs, and test-doc to use parameterized commands and streamlined documentation generation. |
| README.md | Added a Codecov badge to the header section. |
Comments suppressed due to low confidence (1)
justfile:26
- There is potential ambiguity between 'package' and 'main_crate' in the recipe header and subsequent commands. Consider aligning the parameter name used throughout to ensure consistent and clear reference across the commands.
check-if-published package=main_crate: (assert-cmd 'jq')
| cargo doc {{features_flag}} --no-deps | ||
|
|
||
| # Test documentation generation | ||
| test-doc: (docs '') |
There was a problem hiding this comment.
Consider adding an inline comment explaining that the 'test-doc' recipe intentionally reuses the 'docs' command for generating test documentation, which can help future maintainers understand the design decision.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #32 +/- ##
=======================================
Coverage 70.43% 70.43%
=======================================
Files 4 4
Lines 345 345
Branches 345 345
=======================================
Hits 243 243
Misses 83 83
Partials 19 19 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
## 🤖 New release * `tilejson`: 0.4.2 -> 0.4.3 (✓ API compatible changes) <details><summary><i><b>Changelog</b></i></summary><p> <blockquote> ## [0.4.3](v0.4.2...v0.4.3) - 2025-06-16 ### Other - set release-plz CI ([#43](#43)) ([#36](#36)) - minor justfile and precommit adjustments ([#35](#35)) - Ensure create has not been published ([#34](#34)) - ci updates ([#33](#33)) - more ci updates ([#32](#32)) - minor dependabot settings update - Rework CI and justfile ([#31](#31)) - Sort justfiles alphabetically ([#30](#30)) <a name="v0.4.2"></a> ### v0.4.2 (2025-03-03) * Update dependencies, set MSRV to 1.78, and some internal cleanup <a name="v0.4.1"></a> ### v0.4.1 (2022-12-08) * Add `Bounds::from` for `(f64, f64, f64, f64)` tuple. Same for `f32` and `i32`. * Add `Center::from` for `(f64, f64, u8)` and `(f32, f32, u8)` tuples. * A few clippy-related fixes <a name="v0.4.0"></a> ### v0.4.0 (2022-11-19) * Switch all `HashMap` to `BTreeMap` for consistent serialization ordering <a name="v0.3.4"></a> ### v0.3.4 (2022-11-15) * Add proper `Error` implementation to `Bounds` and `Center` parsing errors <a name="v0.3.3"></a> ### v0.3.3 (2022-11-07) * Add `Display` with precision support for `Bounds` and `Center` structs <a name="v0.3.2"></a> ### v0.3.2 (2022-10-30) * Add `Bounds::from` for `[f64; 4]`, `[f32; 4]`, `[i32; 4]` * Add `Bounds::try_from` now also supports `&[f64]`, `&[f32]`, `&[i32]` in addition to `Vec<f64>` <a name="v0.3.1"></a> ### v0.3.1 (2022-05-29) * Add `Bounds::MAX` to create a maximum -180..180, -90..90 value. * Add `Bounds::MAX_TILED` to create a maximum allowed for vector tiles per spec. * Implement `Add` and `AddAssign` on `Bounds` <a name="v0.3.0"></a> ### v0.3.0 (2022-05-25) **ATTENTION:** This release contains many breaking changes. See [README](README.md) for usage examples. * Migrate to Rust 2021 edition * update docs to match v3.0.0 spec * add `fillzoom` field per v3.0.0 spec * add `Center` and `Bounds` structs instead of arrays * both support `FromStr` trait * add `VectorLayer` struct and the `vector_layer` field * Remove builder pattern because `TileJSON` is writable * Add `other` fields for any unknown fields in root and vector layers * Restructure instantiation: * use `tilejson!{ source }` macro to create `TileJSON` objects, with any number of the optional `field: value` pairs. * use `set_missing_defaults()` to replace all missing values with their defaults (only if the spec defines it) * Remove `id` field because it is not supported by the spec <a name="v0.2.4"></a> ### v0.2.4 (2021-10-11) #### Bug Fixes * use String instead of &'static str for `tilejson` field (#7) ([25b325c](25b325c)) <a name="v0.2.3"></a> ### v0.2.3 (2021-10-10) #### Bug Fixes * skip serializing if Option is None (#6) (h/t @jaspervercnocke) ([149339c](149339c)) <a name="0.2.2"></a> ### v0.2.2 (2020-09-12) #### Features * change type of bounds to f32 instead of i32 (#1) (h/t @jaspervercnocke) ([a7cffa8](a7cffa8)) #### Bug Fixes * use crate pub ([09f051a](09f051a)) </blockquote> </p></details> --- This PR was generated with [release-plz](https://github.com/release-plz/release-plz/).
No description provided.