-
Notifications
You must be signed in to change notification settings - Fork 8
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
chore: release v0.6.2 #156
Conversation
WalkthroughThe pull request includes updates to the Changes
Possibly related PRs
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
crates/augurs-prophet/CHANGELOG.md (1)
12-14
: Consider adding more context to the fix description.While the fix description "use OUT_DIR instead of CARGO_MANIFEST_DIR" is accurate, it would be more helpful to briefly explain why this change was necessary or what issues it resolves.
Consider expanding it to something like:
-use OUT_DIR instead of CARGO_MANIFEST_DIR +use OUT_DIR instead of CARGO_MANIFEST_DIR for more reliable build artifacts location
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (2)
Cargo.toml
(2 hunks)crates/augurs-prophet/CHANGELOG.md
(1 hunks)
🔇 Additional comments (4)
Cargo.toml (2)
21-21
: LGTM: Workspace version bump to 0.6.2
Version bump follows semantic versioning and aligns with the release objectives.
32-41
: LGTM: Consistent version updates across workspace dependencies
All internal crate dependencies are correctly updated to version 0.6.2, maintaining version consistency across the workspace.
Let's verify that all workspace crates are included in this version update:
✅ Verification successful
Missing dependency: augurs-testing crate not included in dependencies
The workspace contains augurs-testing
crate at crates/augurs-testing/
but it's not listed in the dependencies section of Cargo.toml
. Let's verify if this is intentional by checking its role in the workspace:
Version updates are correctly applied - augurs-testing intentionally excluded
The version consistency is verified and correct. The augurs-testing
crate is intentionally not included in the dependencies because:
- It's marked with
publish = false
indicating it's an internal testing utility - It's described as "Test framework and data for the augurs library"
- It doesn't appear as a dependency in the main Cargo.toml, which is correct for test-only crates
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify all workspace crates are included in the version update
# and that no crates are missing from the dependencies list.
# Test: List all crates in the workspace and compare with dependencies
echo "Workspace crates:"
fd -t d "^augurs-" "crates/" --max-depth 1
echo -e "\nListed dependencies:"
rg "^augurs-.*= \{ version" Cargo.toml
Length of output: 1136
Script:
#!/bin/bash
# Check if augurs-testing is a dev-dependency or test-only crate
echo "Checking Cargo.toml for dev-dependencies:"
rg "augurs-testing.*version" Cargo.toml
echo -e "\nChecking augurs-testing's Cargo.toml:"
cat crates/augurs-testing/Cargo.toml
Length of output: 677
crates/augurs-prophet/CHANGELOG.md (2)
10-11
: LGTM: Version entry follows the standard format.
The version entry is properly formatted with:
- Correct version number (0.6.2)
- Valid GitHub compare link
- ISO 8601 date format
16-18
: LGTM: Other section is well documented.
The addition of the Prophet WASM example is properly documented with a reference to the relevant PR (#152).
🤖 New release
augurs
: 0.6.1 -> 0.6.2augurs-changepoint
: 0.6.1 -> 0.6.2augurs-core
: 0.6.1 -> 0.6.2augurs-clustering
: 0.6.1 -> 0.6.2augurs-dtw
: 0.6.1 -> 0.6.2augurs-ets
: 0.6.1 -> 0.6.2augurs-mstl
: 0.6.1 -> 0.6.2augurs-forecaster
: 0.6.1 -> 0.6.2augurs-outlier
: 0.6.1 -> 0.6.2augurs-prophet
: 0.6.1 -> 0.6.2 (✓ API compatible changes)augurs-seasons
: 0.6.1 -> 0.6.2Changelog
augurs
augurs-changepoint
augurs-core
augurs-clustering
augurs-dtw
augurs-ets
augurs-mstl
augurs-forecaster
augurs-outlier
augurs-prophet
augurs-seasons
This PR was generated with release-plz.
Summary by CodeRabbit
New Features
0.6.2
.Bug Fixes
CARGO_MANIFEST_DIR
withOUT_DIR
in the Prophet package.Documentation
0.6.2
.