-
Notifications
You must be signed in to change notification settings - Fork 21
chore: release v0.9.1 #249
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
Conversation
WalkthroughThis pull request involves a version update for the Augurs project, incrementing the main package and its dependencies from version 0.9.0 to 0.9.1. The changes primarily affect the Changes
Possibly related PRs
Poem
π Recent review detailsConfiguration used: CodeRabbit UI π Files selected for processing (2)
π§ Files skipped from review as they are similar to previous changes (2)
πͺ§ 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
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
β Files skipped from review due to trivial changes (1)
- CHANGELOG.md
β° Context from checks skipped due to timeout of 90000ms (23)
- GitHub Check: windows (windows-latest, x86)
- GitHub Check: macos (macos-14, aarch64)
- GitHub Check: windows (windows-latest, x64)
- GitHub Check: macos (macos-13, x86_64)
- GitHub Check: linux (ubuntu-22.04, ppc64le)
- GitHub Check: musllinux (ubuntu-22.04, armv7)
- GitHub Check: linux (ubuntu-22.04, s390x)
- GitHub Check: musllinux (ubuntu-22.04, aarch64)
- GitHub Check: linux (ubuntu-22.04, armv7)
- GitHub Check: musllinux (ubuntu-22.04, x86)
- GitHub Check: Clippy
- GitHub Check: linux (ubuntu-22.04, aarch64)
- GitHub Check: Test Book
- GitHub Check: Tests
- GitHub Check: musllinux (ubuntu-22.04, x86_64)
- GitHub Check: linux (ubuntu-22.04, x86)
- GitHub Check: Rustfmt
- GitHub Check: linux (ubuntu-22.04, x86_64)
- GitHub Check: Check
- GitHub Check: Run benchmarks
- GitHub Check: JS tests
- GitHub Check: Python tests
- GitHub Check: Prophet WASMStan component
π Additional comments (1)
Cargo.toml (1)
21-21
: LGTM! Version bump aligns with release objectives.The patch version increment (0.9.0 -> 0.9.1) follows semantic versioning conventions.
augurs-changepoint = { version = "0.9.1", path = "crates/augurs-changepoint" } | ||
augurs-clustering = { version = "0.9.1", path = "crates/augurs-clustering" } | ||
augurs-core = { version = "0.9.1", path = "crates/augurs-core" } | ||
augurs-dtw = { version = "0.9.1", path = "crates/augurs-dtw" } | ||
augurs-ets = { version = "0.9.1", path = "crates/augurs-ets" } | ||
augurs-forecaster = { version = "0.9.1", path = "crates/augurs-forecaster" } | ||
augurs-mstl = { version = "0.9.1", path = "crates/augurs-mstl" } | ||
augurs-outlier = { version = "0.9.1", path = "crates/augurs-outlier" } | ||
augurs-prophet = { version = "0.9.1", path = "crates/augurs-prophet" } | ||
augurs-seasons = { version = "0.9.1", 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
Missing changelog entries for version 0.9.1
While all packages are being updated to version 0.9.1 in Cargo.toml, only augurs-prophet
has a corresponding changelog entry. Please add changelog entries for version 0.9.1 in:
- Root CHANGELOG.md for all updated packages
- Package-specific CHANGELOG.md files in:
- crates/augurs-changepoint/
- crates/augurs-clustering/
- crates/augurs-core/
- crates/augurs-dtw/
- crates/augurs-ets/
- crates/augurs-forecaster/
- crates/augurs-mstl/
- crates/augurs-outlier/
- crates/augurs-seasons/
π Analysis chain
LGTM! Dependency versions are consistently updated.
All internal workspace dependencies are correctly updated to match the new version 0.9.1, maintaining consistency across the project.
Let's verify that each updated package has corresponding changelog entries:
π Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check for changelog entries for each updated package
# List of packages to check
packages=(
"augurs-changepoint"
"augurs-clustering"
"augurs-core"
"augurs-dtw"
"augurs-ets"
"augurs-forecaster"
"augurs-mstl"
"augurs-outlier"
"augurs-prophet"
"augurs-seasons"
)
echo "Checking for changelog entries in version 0.9.1..."
for pkg in "${packages[@]}"; do
echo "Checking $pkg..."
# Check main CHANGELOG.md
if rg -q "## \[0\.9\.1\]" "CHANGELOG.md"; then
rg -A 10 "## \[0\.9\.1\]" "CHANGELOG.md"
fi
# Check package-specific CHANGELOG.md
if [ -f "crates/$pkg/CHANGELOG.md" ]; then
if rg -q "## \[0\.9\.1\]" "crates/$pkg/CHANGELOG.md"; then
rg -A 10 "## \[0\.9\.1\]" "crates/$pkg/CHANGELOG.md"
fi
fi
done
Length of output: 2663
Script:
#!/bin/bash
# Check existence and content of changelog files
# First check root CHANGELOG.md
echo "=== Root CHANGELOG.md ==="
if [ -f "CHANGELOG.md" ]; then
echo "Found root CHANGELOG.md"
head -n 20 "CHANGELOG.md"
else
echo "No root CHANGELOG.md found"
fi
echo -e "\n=== Package CHANGELOGs ==="
for pkg in augurs-{changepoint,clustering,core,dtw,ets,forecaster,mstl,outlier,prophet,seasons}; do
echo -e "\n--- $pkg ---"
if [ -f "crates/$pkg/CHANGELOG.md" ]; then
echo "Found CHANGELOG.md"
head -n 20 "crates/$pkg/CHANGELOG.md"
else
echo "No CHANGELOG.md found"
fi
done
Length of output: 10414
bffccfd
to
5992142
Compare
f3b19cc
to
80d9ea5
Compare
ecf6c82
to
47fb6a1
Compare
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
π§Ή Nitpick comments (1)
CHANGELOG.md (1)
10-16
: Review the 'augurs-prophet' Changelog SectionThe new changelog entry for
augurs-prophet
is clear and follows the expected formatβwith a version bump to 0.9.1, a proper comparison link, and a correctly formatted release date. The sub-section "Other" neatly groups the additional changes like the JS bindings and dependency updates.Nitpick: The static analysis tool suggests using "Others" instead of "Other" for the sub-heading. Please verify if the projectβs changelog style requires the plural form. If so, consider applying the following small diff:
-### Other +### Othersπ§° Tools
πͺ LanguageTool
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-12 ### Other - Add JS bindings for Prophet make_futu...(OTHER_OTHERS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- Cargo.toml
π§° Additional context used
πͺ LanguageTool
CHANGELOG.md
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-12 ### Other - Add JS bindings for Prophet make_futu...
(OTHER_OTHERS)
π Additional comments (1)
CHANGELOG.md (1)
17-21
: Approve the 'augurs-outlier' Changelog EntryThe entry for
augurs-outlier
is well-structured. It correctly displays the version update and clearly identifies the addition of setters for parameters under the "Added" sub-heading. The compare link and date are consistent with the rest of the changelog.
47fb6a1
to
1381561
Compare
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
π§Ή Nitpick comments (1)
CHANGELOG.md (1)
12-12
: Nitpick: Consider Revising the Heading for ConsistencyThe current header "### Other" might be more grammatically consistent if changed to "### Others," as suggested by static analysis. If it aligns with your changelog style, you might want to update it.
Proposed diff:
-### Other +### Othersπ§° Tools
πͺ LanguageTool
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-17 ### Other - Add JS bindings for Prophet make_futu...(OTHER_OTHERS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- Cargo.toml
π§° Additional context used
πͺ LanguageTool
CHANGELOG.md
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-17 ### Other - Add JS bindings for Prophet make_futu...
(OTHER_OTHERS)
π Additional comments (3)
CHANGELOG.md (3)
10-11
: Release Entry for augurs-prophet is Correctly FormattedThe new changelog entry for the
augurs-prophet
module documents the version update to0.9.1
, includes an appropriate comparison link, and has a consistent release date.
13-15
: Dependency Updates and JS Bindings Entry are Well DocumentedThe bullet points in this section clearly list the addition of JS bindings for the
make_future_dataframe
function as well as the dependency updates forureq
(from 2.10.1 to 3.0.0) andwasmtime
(from 28 to 29). They properly reference their corresponding issue numbers.
17-20
: augurs-outlier Section is Consistent and InformativeThe new section for
augurs-outlier
is well formatted. It correctly documents the version bump to0.9.1
and notes the addition of setters for parameters with an appropriate reference. This follows the established format and maintains consistency across the changelog.
1381561
to
a05f08b
Compare
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
π§Ή Nitpick comments (1)
CHANGELOG.md (1)
10-16
: Document New Release Entry foraugurs-prophet
The new section for
augurs-prophet
version 0.9.1 is well structured. It clearly includes the release link, release date, and detailed changelog items (JS bindings addition and dependency updates).Consider revising the header on line 12 from "### Other" to "### Others" for better grammatical consistency.
An optional diff suggestion:
-### Other +### Othersπ§° Tools
πͺ LanguageTool
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-17 ### Other - Add JS bindings for Prophet make_futu...(OTHER_OTHERS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- Cargo.toml
π§° Additional context used
πͺ LanguageTool
CHANGELOG.md
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-17 ### Other - Add JS bindings for Prophet make_futu...
(OTHER_OTHERS)
π Additional comments (1)
CHANGELOG.md (1)
17-21
: Proper Addition foraugurs-outlier
ModuleThe changelog entry for
augurs-outlier
version 0.9.1 is concise and clearly states the addition of setters for parameters. This entry follows the established format and adds meaningful context to the release notes.
a05f08b
to
53d5e9b
Compare
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
π§Ή Nitpick comments (1)
CHANGELOG.md (1)
12-15
: Note on the 'Other' SectionThe "### Other" section neatly documents the additional changes such as:
- JS bindings for
Prophet make_future_dataframe
(#257)- Dependency updates for ureq (2.10.1 β 3.0.0) and wasmtime (28 β 29)
As a minor nitpick, the static analysis hint suggests that βothersβ might be more grammatically appropriate in some contexts. If preferred for consistency with your style guidelines, consider renaming the header to β### Others.β
π§° Tools
πͺ LanguageTool
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-17 ### Other - Add JS bindings for Prophet make_futu...(OTHER_OTHERS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- Cargo.toml
π§° Additional context used
πͺ LanguageTool
CHANGELOG.md
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-17 ### Other - Add JS bindings for Prophet make_futu...
(OTHER_OTHERS)
π Additional comments (2)
CHANGELOG.md (2)
10-11
: Review: Augurs-Prophet Release HeaderThe section header for
augurs-prophet
version 0.9.1 is formatted well with a clear version link and release date. Please verify that the link accurately reflects the difference between version 0.9.0 and 0.9.1.
17-21
: Review: Augurs-Outlier Addition SectionThe section for
augurs-outlier
version 0.9.1 is clear and concise. It correctly introduces the release update along with the new addition of setters for parameters (#253). The use of the "### Added" category is appropriate and consistent with changelog conventions.
53d5e9b
to
696f788
Compare
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
π§Ή Nitpick comments (1)
CHANGELOG.md (1)
10-16
: Newaugurs-prophet
Section & Dependency UpdatesThe new section for
augurs-prophet
version 0.9.1 is well formatted. It includes the version comparison link and the release date, and it documents:
- The addition of JS bindings for the
make_future_dataframe
function.- The dependency updates for both the
ureq
requirement (from 2.10.1 to 3.0.0) and thewasmtime
requirement (from 28 to 29).One minor nitpick: the heading "Other" on line 12 has been flagged by static analysis for possibly using a plural form ("Others") if that improves consistency with the change log style. This is optional and you may retain the current style based on your project guidelines.
π§° Tools
πͺ LanguageTool
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-25 ### Other - Add JS bindings for Prophet make_futu...(OTHER_OTHERS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- Cargo.toml
π§° Additional context used
πͺ LanguageTool
CHANGELOG.md
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-25 ### Other - Add JS bindings for Prophet make_futu...
(OTHER_OTHERS)
π Additional comments (1)
CHANGELOG.md (1)
17-21
: Newaugurs-outlier
Section AdditionThe changelog now includes an entry for
augurs-outlier
version 0.9.1 with a clear title and release date along with the "Added" category. The description accurately documents the addition of setters for parameters and correctly references issue [#253].
696f788
to
739aaed
Compare
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
π§Ή Nitpick comments (1)
CHANGELOG.md (1)
12-16
: Evaluation of "Other" Section for augurs-prophet
The "Other" category correctly lists the JS bindings addition and dependency updates with clear references to the related issues. One minor nitpick: static analysis suggests that βOthersβ (plural) might be more appropriate than βOtherβ in this context for consistency and clarity. Consider updating if it aligns with the style guidelines of your changelog.π§° Tools
πͺ LanguageTool
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-26 ### Other - Add JS bindings for Prophet make_futu...(OTHER_OTHERS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- Cargo.toml
π§° Additional context used
πͺ LanguageTool
CHANGELOG.md
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-03-26 ### Other - Add JS bindings for Prophet make_futu...
(OTHER_OTHERS)
π Additional comments (2)
CHANGELOG.md (2)
10-11
: Review of augurs-prophet Section Header
The header for theaugurs-prophet
module is formatted consistently with other sections and correctly includes the version, comparison link, and release date. Please double-check the release date for accuracy if needed.
17-21
: Review of augurs-outlier Section
The newly added section foraugurs-outlier
is well-structured and includes the version update, comparison link, release date, and a clear bullet point for the addition of setters for parameters with the appropriate issue reference. Everything appears to be in order.
739aaed
to
ab12045
Compare
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
π§Ή Nitpick comments (1)
CHANGELOG.md (1)
12-16
: Consider Renaming the "Other" HeadingThe header on line 12 is currently labeled as "### Other". Based on the static analysis suggestion and for better clarity (since multiple items are listed), consider renaming it to "### Others".
Below is a proposed diff for this change:
-### Other +### Othersπ§° Tools
πͺ LanguageTool
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-04-01 ### Other - Add JS bindings for Prophet make_futu...(OTHER_OTHERS)
π Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π Files selected for processing (2)
CHANGELOG.md
(1 hunks)Cargo.toml
(2 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- Cargo.toml
π§° Additional context used
πͺ LanguageTool
CHANGELOG.md
[misspelling] ~12-~12: It seems that the plural noun βothersβ fits better in this context.
Context: ...ugurs-prophet-v0.9.1) - 2025-04-01 ### Other - Add JS bindings for Prophet make_futu...
(OTHER_OTHERS)
π Additional comments (2)
CHANGELOG.md (2)
10-11
: Review theaugurs-prophet
Release Entry FormattingThe entry for
augurs-prophet
version0.9.1
(lines 10-11) is clear and follows the expected format with a proper comparison link and release date. Please double-check that the URL accurately reflects the intended comparison.
17-21
: Review theaugurs-outlier
Changelog EntryThe new entry for
augurs-outlier
version0.9.1
(lines 17-21) is well-formatted, featuring the correct comparison link, release date, and a concise note on the addition of setters for parameters ([#253]). This entry is consistent with the style used elsewhere in the changelog.
ab12045
to
bf9d928
Compare
bf9d928
to
d3c0c62
Compare
π€ New release
augurs-core
: 0.9.0 -> 0.9.1augurs-changepoint
: 0.9.0 -> 0.9.1augurs-clustering
: 0.9.0 -> 0.9.1 (β API compatible changes)augurs-dtw
: 0.9.0 -> 0.9.1augurs-mstl
: 0.9.0 -> 0.9.1augurs-ets
: 0.9.0 -> 0.9.1 (β API compatible changes)augurs-forecaster
: 0.9.0 -> 0.9.1augurs-outlier
: 0.9.0 -> 0.9.1 (β API compatible changes)augurs-prophet
: 0.9.0 -> 0.9.1 (β API compatible changes)augurs-seasons
: 0.9.0 -> 0.9.1augurs
: 0.9.0 -> 0.9.1Changelog
This PR was generated with release-plz.
Summary by CodeRabbit
Chores
Documentation