-
Notifications
You must be signed in to change notification settings - Fork 7
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 #154
Conversation
WalkthroughThe pull request involves 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: 1
🧹 Outside diff range and nitpick comments (1)
crates/augurs-prophet/CHANGELOG.md (1)
12-14
: Consider adding more context to the changelog entry.While the fix is correctly categorized, consider expanding the description to include:
- Why the change from CARGO_MANIFEST_DIR to OUT_DIR was necessary
- What issues this change resolves
- The impact of this change on the build process
This additional context would be valuable for future maintainers.
### Fixed -use OUT_DIR instead of CARGO_MANIFEST_DIR +use OUT_DIR instead of CARGO_MANIFEST_DIR for build artifacts to ensure correct +placement of compiled files and compatibility with cargo's build system
📜 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 (2)
Cargo.toml (1)
21-21
: LGTM: Workspace version update is correct.
Version bump from 0.6.1 to 0.6.2 aligns with the release objectives.
crates/augurs-prophet/CHANGELOG.md (1)
10-11
: LGTM: Version entry follows the standard format.
The version number, comparison link, and date are correctly formatted according to Keep a Changelog conventions.
augurs-changepoint = { version = "0.6.2", path = "crates/augurs-changepoint" } | ||
augurs-clustering = { version = "0.6.2", path = "crates/augurs-clustering" } | ||
augurs-core = { version = "0.6.2", path = "crates/augurs-core" } | ||
augurs-dtw = { version = "0.6.2", path = "crates/augurs-dtw" } | ||
augurs-ets = { version = "0.6.2", path = "crates/augurs-ets" } | ||
augurs-forecaster = { version = "0.6.2", path = "crates/augurs-forecaster" } | ||
augurs-mstl = { version = "0.6.2", path = "crates/augurs-mstl" } | ||
augurs-outlier = { version = "0.6.2", path = "crates/augurs-outlier" } | ||
augurs-prophet = { version = "0.6.2", path = "crates/augurs-prophet" } | ||
augurs-seasons = { version = "0.6.2", path = "crates/augurs-seasons" } |
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.
💡 Codebase verification
Note: augurs-testing
dependency is missing version specification
The verification revealed that while most augurs-* dependencies are correctly updated to version 0.6.2, the augurs-testing
crate is specified with only a path and no version:
augurs-testing = { path = "crates/augurs-testing" }
For consistency with other workspace dependencies, it should also include the version "0.6.2".
🔗 Analysis chain
LGTM: Dependency versions are consistently updated.
All augurs-* dependencies have been correctly updated to version 0.6.2, maintaining consistency across the workspace.
Let's verify that no dependencies were missed in this update:
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check for any augurs-* dependencies that might still reference version 0.6.1
# Test: Search for any remaining 0.6.1 versions in Cargo.toml files
# Expect: No matches as all should be updated to 0.6.2
echo "Checking for any remaining 0.6.1 versions..."
rg '"0.6.1"' -g 'Cargo.toml'
# Test: Verify all augurs-* crates in workspace are included in dependencies
# Expect: All workspace crates should have corresponding dependency entries
echo "Checking for any missing augurs-* crates..."
fd '^augurs-.*' crates -t d -x echo "Checking if {} is in dependencies..." \; -x rg "{}" Cargo.toml
Length of output: 1693
🤖 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
, ensuring consistency across the application.Bug Fixes
augurs-prophet
by utilizingOUT_DIR
, improving compatibility with build systems.Documentation
augurs-prophet
to reflect recent changes and improvements.