Update CHANGELOG.md for v0.6.30#1497
Conversation
WalkthroughBumped project version to v0.6.30 across Makefile, Helm chart, and OpenAPI docs. Updated CHANGELOG with new v0.6.30 entry and replaced a prior single refactor note with three specific items. No API endpoints or logic changed. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Pre-merge checks (3 passed)✅ Passed checks (3 passed)
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. ✨ Finishing touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
CHANGELOG.md (2)
10-15: Changelog entry added for v0.6.30; minor Markdown nits.Consider formatting the raw PR URLs to satisfy MD034 (no-bare-urls).
Apply:
- - Refactor housekeeping logic for simpler candidate fetching by @hackerwins in https://github.com/yorkie-team/yorkie/pull/1492 - - Enhance QueryMonitor with detailed collections and filter by @hackerwins in https://github.com/yorkie-team/yorkie/pull/1494 - - Count only successfully compacted documents by @hackerwins in https://github.com/yorkie-team/yorkie/pull/1495 - - Add project-level RemoveOnDetach setting by @emplam27 in https://github.com/yorkie-team/yorkie/pull/1496 + - Refactor housekeeping logic for simpler candidate fetching by @hackerwins in <https://github.com/yorkie-team/yorkie/pull/1492> + - Enhance QueryMonitor with detailed collections and filter by @hackerwins in <https://github.com/yorkie-team/yorkie/pull/1494> + - Count only successfully compacted documents by @hackerwins in <https://github.com/yorkie-team/yorkie/pull/1495> + - Add project-level RemoveOnDetach setting by @emplam27 in <https://github.com/yorkie-team/yorkie/pull/1496>
10-10: Style consistency (optional).Recent sections mix
[v0.x.y]and[0.x.y]. If you prefer thevprefix, consider standardizing for 0.6.x entries in a follow-up.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
CHANGELOG.md(1 hunks)Makefile(1 hunks)api/docs/yorkie.base.yaml(1 hunks)api/docs/yorkie/v1/admin.openapi.yaml(1 hunks)api/docs/yorkie/v1/cluster.openapi.yaml(1 hunks)api/docs/yorkie/v1/resources.openapi.yaml(1 hunks)api/docs/yorkie/v1/yorkie.openapi.yaml(1 hunks)build/charts/yorkie-cluster/Chart.yaml(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-07-09T01:45:42.906Z
Learnt from: KMSstudio
PR: yorkie-team/yorkie#1372
File: cmd/yorkie/signup.go:0-0
Timestamp: 2025-07-09T01:45:42.906Z
Learning: In the Yorkie CLI codebase (cmd/yorkie/), global variables like `username`, `password`, `rpcAddr`, and `insecure` are declared in `login.go` at package level and are reused across different commands (like `signup.go`) within the same `main` package, avoiding unnecessary duplication.
Applied to files:
Makefile
🪛 markdownlint-cli2 (0.17.2)
CHANGELOG.md
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
15-15: Bare URL used
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🔇 Additional comments (7)
Makefile (1)
1-1: Version bump aligned and verified — approving.YORKIE_VERSION set to 0.6.30 (Makefile). Chart and OpenAPI docs updated (build/charts/yorkie-cluster/Chart.yaml, api/docs/**); no lingering prerelease/old-version tags found (CHANGELOG contains prior v0.6.29 entry).
api/docs/yorkie/v1/cluster.openapi.yaml (1)
6-6: OpenAPI info.version updated.v0.6.30 set; no API surface change implied.
api/docs/yorkie.base.yaml (1)
5-5: Base OpenAPI version bumped.v0.6.30 set; consistent with per-service specs.
api/docs/yorkie/v1/admin.openapi.yaml (1)
6-6: Admin OpenAPI version bumped.v0.6.30 set; endpoints unchanged.
api/docs/yorkie/v1/resources.openapi.yaml (1)
6-6: Resources OpenAPI version bumped.v0.6.30 set; looks good.
api/docs/yorkie/v1/yorkie.openapi.yaml (1)
6-6: Yorkie service OpenAPI version bumped.v0.6.30 set; consistent across specs.
build/charts/yorkie-cluster/Chart.yaml (1)
14-15: Helm chart version/appVersion synced to 0.6.30.Matches Makefile and docs.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1497 +/- ##
==========================================
- Coverage 48.64% 48.61% -0.03%
==========================================
Files 180 180
Lines 22380 22380
==========================================
- Hits 10886 10880 -6
- Misses 10572 10576 +4
- Partials 922 924 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
What this PR does / why we need it:
Update CHANGELOG.md for v0.6.30
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Additional documentation:
Checklist:
Summary by CodeRabbit
New Features
Documentation
Chores