[docs] Changelogs for release series v0.35.x #1347
Conversation
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. WalkthroughRemoved a top-line placeholder from the changelog template, added a "Full Changelog" link in the Development/Testing/CI/CD section of the template, and added three new changelog files for v0.35.0, v0.35.1, and v0.35.2. All edits are documentation-only. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Summary of Changes
Hello @NickVolynkin, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!
This pull request primarily focuses on updating and expanding the project's documentation by adding comprehensive changelog entries for the v0.34.x release series, spanning from v0.34.0 through v0.34.5, and introducing the initial changelog for v0.35.0-beta.1. It also includes minor refinements to the changelog template. The main purpose is to ensure that all changes, new features, security updates, and bug fixes across these versions are clearly documented and easily accessible to users and developers, providing a complete and accurate historical record of the project's evolution.
Highlights
- Changelog Template Enhancement: The
template.mdfile was updated to include a 'Release description' section and a 'Full Changelog' link, standardizing the structure for future release notes. - Comprehensive v0.34.x Changelog Additions: New, detailed changelog files were added for all patch versions of the
v0.34.xseries (v0.34.0, v0.34.1, v0.34.2, v0.34.3, v0.34.4, and v0.34.5), providing a clear history of features, fixes, and updates for each specific release. - Introduction of v0.35.0-beta.1 Changelog: A new changelog file for
v0.35.0-beta.1was introduced, documenting significant major features, security improvements, various bug fixes, and updated dependencies for the upcoming release.
Using Gemini Code Assist
The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.
Invoking Gemini
You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
Customization
To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.
Limitations & Feedback
Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.
You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.
Footnotes
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
There was a problem hiding this comment.
Code Review
This pull request adds changelog files for the v0.34.x and v0.35.x release series. The changes are well-structured and provide a good overview of the new features, fixes, and other updates. I've found a few minor inconsistencies and typos in the new markdown files and have left specific comments with suggestions for correction. These are mostly related to formatting to ensure consistency across all changelog entries.
|
|
||
| --- | ||
|
|
||
| **Full Changelog**: **Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 |
There was a problem hiding this comment.
The phrase "Full Changelog:" is duplicated. This should be corrected to avoid propagating the error to new changelogs created from this template.
| **Full Changelog**: **Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 | |
| **Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 |
docs/changelogs/v0.34.0.md
Outdated
| * [cozystack-api] Fix an error in updating lists in Cozystack objects, which resulted in message "Warning: resource ... is missing the kubectl.kubernetes.io/last-applied-configuration annotation". (@kvaps in https://github.com/cozystack/cozystack/pull/1171) | ||
| * [cozystack-api] Disable `strategic-json-patch` support. (@kvaps in https://github.com/cozystack/cozystack/pull/1179) | ||
| * [cozystack-api] Fix non-existing OpenAPI references. (@kvaps in https://github.com/cozystack/cozystack/pull/1208) | ||
| * [dashboard] Fix the code for removing dashboard comments which used to mistakenly remove shebang from `cloudInit` scripts. (@kvaps in https://github.com/cozystack/cozystack/pull/1175). |
There was a problem hiding this comment.
There is an extra period at the end of this line, which is inconsistent with the formatting of other entries in this file. For consistency, it should be removed.
| * [dashboard] Fix the code for removing dashboard comments which used to mistakenly remove shebang from `cloudInit` scripts. (@kvaps in https://github.com/cozystack/cozystack/pull/1175). | |
| * [dashboard] Fix the code for removing dashboard comments which used to mistakenly remove shebang from `cloudInit` scripts. (@kvaps in https://github.com/cozystack/cozystack/pull/1175) |
docs/changelogs/v0.34.0.md
Outdated
| * [oidc] Enable deleting Keycloak service. (@klinch0 in https://github.com/cozystack/cozystack/pull/1178) | ||
| * [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) | ||
| * [nats] Fix a typo in the application template. (@klinch0 in https://github.com/cozystack/cozystack/pull/1195) | ||
| * [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) |
There was a problem hiding this comment.
The preposition "in" is missing between the author's handle and the pull request link. This is inconsistent with other entries in the changelog.
| * [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) | |
| * [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/1204) |
docs/changelogs/v0.35.0.md
Outdated
| * [platform] Autodetect if node ports should be assigned to load balancer services. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1271) | ||
| * [seaweedfs] Enable [integration with remote filer endpoints](https://cozystack.io/docs/operations/stretched/seaweedfs-multidc/) by adding new `Client` topology. (@kvaps in https://github.com/cozystack/cozystack/pull/1239) | ||
| * [cozystack-api] Show default values from the OpenAPI spec in the application resources. (@kvaps in https://github.com/cozystack/cozystack/pull/1241) | ||
| * [cozystack-api] Configure dynamic API using Custom Resources. (@klinch in https://github.com/cozystack/cozystack/pull/1230) |
There was a problem hiding this comment.
There appears to be a typo in the author's username. It's written as @klinch, but in other places in this PR and repository, it's @klinch0. Please verify and correct for consistency.
| * [cozystack-api] Configure dynamic API using Custom Resources. (@klinch in https://github.com/cozystack/cozystack/pull/1230) | |
| * [cozystack-api] Configure dynamic API using Custom Resources. (@klinch0 in https://github.com/cozystack/cozystack/pull/1230) |
| ## Development, Testing, and CI/CD | ||
|
|
||
| * [tests] Add tests for S3 buckets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1283) | ||
| * [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) |
There was a problem hiding this comment.
There is an extra space after "in" which appears to be a minor typo.
| * [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) | |
| * [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (23)
docs/changelogs/v0.34.5.md (2)
8-8: Replace bare PR URLs with Markdown links to satisfy markdownlint (MD034).Convert the raw URLs to linked PR numbers for consistency and lint compliance.
-* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1300, backported in https://github.com/cozystack/cozystack/pull/1303) +* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in [#1300](https://github.com/cozystack/cozystack/pull/1300), backported in [#1303](https://github.com/cozystack/cozystack/pull/1303))
10-10: Wrap "Full Changelog" URL in a Markdown link.This avoids MD034 and aligns with the updated template style.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5 +**Full Changelog**: [v0.34.4...v0.34.5](https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5)docs/changelogs/v0.34.1.md (2)
12-13: Replace bare PR/commit URLs with Markdown links.Improves readability and satisfies markdownlint (MD034).
-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) +* [kubernetes] Fix regression in `volumesnapshotclass` installation from [#1203](https://github.com/cozystack/cozystack/pull/1203). (@kvaps in [#1238](https://github.com/cozystack/cozystack/pull/1238)) +* [objectstorage] Fix building objectstorage images. (@kvaps in [a9e9dfc](https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923))
15-15: Wrap "Full Changelog" URL in a Markdown link.-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1 +**Full Changelog**: [v0.34.0...v0.34.1](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1)docs/changelogs/v0.34.2.md (3)
12-12: Replace bare commit URL with a Markdown link.-* [objectstorage] Fix recording image in objectstorage. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637) +* [objectstorage] Fix recording image in objectstorage. (@kvaps in [4d9a838](https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637))
12-12: Possible wording nit: "recording image" — intentional?Please confirm if this should be "building image" or "rebuilding image" instead of "recording image".
14-14: Wrap "Full Changelog" URL in a Markdown link.-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2 +**Full Changelog**: [v0.34.1...v0.34.2](https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2)docs/changelogs/template.md (2)
4-5: Use a clearer placeholder for the release tag URL.Replace the ambiguous v0.. with a conventional placeholder to avoid copy/paste mistakes.
-https://github.com/cozystack/cozystack/releases/tag/v0.. +https://github.com/cozystack/cozystack/releases/tag/vX.Y.Z
21-21: Remove duplicated "Full Changelog" label and avoid bare URL; add placeholders.This is a template, so duplication will propagate. Use a single label and a Markdown link with placeholders.
-**Full Changelog**: **Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 +**Full Changelog**: [vPREV_TAG...vCURR_TAG](https://github.com/cozystack/cozystack/compare/vPREV_TAG...vCURR_TAG)docs/changelogs/v0.34.4.md (1)
7-9: Convert bare URLs to Markdown links across the file (MD034).This improves readability and satisfies markdownlint. Suggest converting PR references to linked numbers and commits to short-SHA links.
-* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1286) -* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in https://github.com/cozystack/cozystack/pull/1287, backported in https://github.com/cozystack/cozystack/pull/1291) +* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in [#1286](https://github.com/cozystack/cozystack/pull/1286)) +* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in [#1287](https://github.com/cozystack/cozystack/pull/1287), backported in [#1291](https://github.com/cozystack/cozystack/pull/1291)) @@ -* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in https://github.com/cozystack/cozystack/pull/1284) -* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in https://github.com/cozystack/cozystack/pull/1290, backported in https://github.com/cozystack/cozystack/pull/1293) +* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in [#1284](https://github.com/cozystack/cozystack/pull/1284)) +* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in [#1290](https://github.com/cozystack/cozystack/pull/1290), backported in [#1293](https://github.com/cozystack/cozystack/pull/1293)) @@ -* [tests] Add tests for S3 buckets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1283, backported in https://github.com/cozystack/cozystack/pull/1292) +* [tests] Add tests for S3 buckets. (@IvanHunters in [#1283](https://github.com/cozystack/cozystack/pull/1283), backported in [#1292](https://github.com/cozystack/cozystack/pull/1292)) @@ -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4 +**Full Changelog**: [v0.34.3...v0.34.4](https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4)Also applies to: 13-14, 19-19, 21-21
docs/changelogs/v0.34.3.md (3)
10-11: Fix KubeVirt capitalization and add terminal periodBrand/style consistency: use “KubeVirt” (capital V). Also align punctuation with surrounding bullets by adding a period.
-* [kubevirt] Fix building Kubevirt CCM (@kvaps in 3c7e256906e1dbb0f957dc3a205fa77a147d419d) +* [kubevirt] Fix building KubeVirt CCM. (@kvaps in https://github.com/cozystack/cozystack/commit/3c7e256906e1dbb0f957dc3a205fa77a147d419d)
7-9: Address markdownlint MD034 (bare URLs) with link markupYour project may accept bare URLs in changelogs; if you want to satisfy markdownlint, wrap URLs in angle brackets or convert to Markdown links.
Examples:
-* [tenant] Fix tenant network policy ... (@klinch0 in https://github.com/cozystack/cozystack/pull/1232, backported in https://github.com/cozystack/cozystack/pull/1272) +* [tenant] Fix tenant network policy ... (@klinch0 in [#1232](https://github.com/cozystack/cozystack/pull/1232), backported in [#1272](https://github.com/cozystack/cozystack/pull/1272))-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3>If you prefer current style, consider suppressing MD034 for changelogs in your linter config instead.
Also applies to: 13-13
1-3: Optional: add a trailing newline at EOFSome tools prefer files ending with a newline. Not critical, but easy to normalize.
docs/changelogs/v0.35.0.md (6)
13-13: Fix author handle: use @klinch0Handle appears elsewhere as @klinch0; align here for consistency.
-* [cozystack-api] Configure dynamic API using Custom Resources. (@klinch in https://github.com/cozystack/cozystack/pull/1230) +* [cozystack-api] Configure dynamic API using Custom Resources. (@klinch0 in https://github.com/cozystack/cozystack/pull/1230)
17-17: Correct project name in link text: cozyvalues-genThe link target is cozyvalues-gen; fix the visible text to match.
-* [apps, cozystack-api] Improve application OpenAPI specs and READMEs by switching to [cozyvalue-gen](https://github.com/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333) +* [apps, cozystack-api] Improve application OpenAPI specs and READMEs by switching to [cozyvalues-gen](https://github.com/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333)
18-18: Use official capitalization: ClickHouseUse the official brand spelling.
-* [clickhouse] Add the Clickhouse Keeper component. (@klinch0 in https://github.com/cozystack/cozystack/pull/1298 and https://github.com/cozystack/cozystack/pull/1320) +* [clickhouse] Add the ClickHouse Keeper component. (@klinch0 in https://github.com/cozystack/cozystack/pull/1298 and https://github.com/cozystack/cozystack/pull/1320)
34-34: Tighten phrasing: “was mistakenly enabled”Minor wording improvement for concision and flow.
-* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature, and it was not designed to work in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301 and https://github.com/cozystack/cozystack/pull/1318) +* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature; it was not designed to work there and was mistakenly enabled. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301 and https://github.com/cozystack/cozystack/pull/1318)
64-64: Capitalize ClickHouse and remove extra space before URLConsistency and minor formatting cleanup.
-* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) +* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete ClickHouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236)
73-74: Address markdownlint MD034 at footer linksWrap the comparison URLs in angle brackets to satisfy the linter without changing visible text.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0-beta.1 -**Since Last Release**: https://github.com/cozystack/cozystack/compare/v0.35.0-alpha.3...v0.35.0-beta.1 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0-beta.1> +**Since Last Release**: <https://github.com/cozystack/cozystack/compare/v0.35.0-alpha.3...v0.35.0-beta.1>docs/changelogs/v0.34.0.md (4)
42-42: Fix pluralization: READMEs (no apostrophe)Documentation plural should be “READMEs”.
-* [applications] Reorder configuration values in application README's for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214) +* [applications] Reorder configuration values in application READMEs for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214)
48-48: Remove duplicated conjunctionClean up “and … and in …”.
-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
50-50: Add missing “in” before PR URLMinor consistency fix.
-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/1204)
87-87: Address markdownlint MD034 at footer linkWrap the URL to satisfy the linter.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (8)
docs/changelogs/template.md(2 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)docs/changelogs/v0.35.0.md(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.5.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: 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)
14-14: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.35.0.md
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
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)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
27-27: Bare URL used
(MD034, no-bare-urls)
27-27: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
29-29: Bare URL used
(MD034, no-bare-urls)
30-30: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
67-67: Bare URL used
(MD034, no-bare-urls)
68-68: Bare URL used
(MD034, no-bare-urls)
69-69: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/template.md
21-21: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.35.0.md
[grammar] ~30-~30: There might be a mistake here.
Context: ...c7e256906e1dbb0f957dc3a205fa77a147d419d) * [virtual-machine] Fix a regression with ...
(QB_NEW_EN)
[style] ~34-~34: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
[grammar] ~37-~37: There might be a mistake here.
Context: ... the volume size limit from 100MB to 30,000MB. (@kvaps in https://github.com/cozystac...
(QB_NEW_EN)
[grammar] ~64-~64: There might be a mistake here.
Context: ...ithub.com//pull/1236) * [dx] When running make commands with `...
(QB_NEW_EN)
[grammar] ~73-~73: There might be a mistake here.
Context: ...zystack/compare/v0.34.0...v0.35.0-beta.1 Since Last Release: https://github.c...
(QB_NEW_EN)
docs/changelogs/v0.34.0.md
[grammar] ~34-~34: There might be a mistake here.
Context: ...ithub.com//pull/1165) * [virtual-machine,vm-instance] Adjusted R...
(QB_NEW_EN)
[grammar] ~36-~36: There might be a mistake here.
Context: ...a3ee5d0ea9e87529c8adc4fcccffabe8782292e) * [cozystack-api] Fix an error with `resou...
(QB_NEW_EN)
[grammar] ~37-~37: There might be a mistake here.
Context: ...r with resourceVersion which resulted in message 'failed to update HelmRelease: ...
(QB_NEW_EN)
[grammar] ~37-~37: There might be a mistake here.
Context: ...ithub.com//pull/1170) * [cozystack-api] Fix an error in updating...
(QB_NEW_EN)
[grammar] ~38-~38: There might be a mistake here.
Context: ...ts in Cozystack objects, which resulted in message "Warning: resource ... is missi...
(QB_NEW_EN)
[grammar] ~40-~40: There might be a mistake here.
Context: ...ithub.com//pull/1208) * [dashboard] Fix the code for removing da...
(QB_NEW_EN)
[grammar] ~44-~44: There might be a mistake here.
Context: ...ithub.com//pull/1172) * [kubernetes] Enable deleting Velero comp...
(QB_NEW_EN)
[grammar] ~45-~45: There might be a mistake here.
Context: ...ystack/pull/1172) * [kubernetes] Enable deleting Velero component from a tenant Kubernet...
(QB_NEW_EN)
[grammar] ~49-~49: There might be a mistake here.
Context: ...ithub.com//pull/1195) * [postgres] Resolve an issue with the vis...
(QB_NEW_EN)
[grammar] ~50-~50: There might be a mistake here.
Context: ...s] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboa...
(QB_NEW_EN)
[grammar] ~85-~85: There might be a mistake here.
Context: ...trictions preventing pushing to ghcr.io with default GitHub token. (@lllamnyp in htt...
(QB_NEW_EN)
🔇 Additional comments (4)
docs/changelogs/v0.34.4.md (1)
5-5: Content LGTM.The structure and messaging are clear, including actionable upgrade guidance for tenant clusters.
docs/changelogs/v0.34.3.md (1)
5-12: Overall content reads wellThe fixes are clear, scoped, and reference sources. Nice work.
docs/changelogs/v0.35.0.md (1)
5-71: Solid, comprehensive changelogWell-structured and easy to scan; references and attributions look good overall.
docs/changelogs/v0.34.0.md (1)
1-11: Nice release intro with clear upgrade guidanceClear warning about regression and direct upgrade path is very helpful.
f430cdf to
3c27f04
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
♻️ Duplicate comments (3)
docs/changelogs/template.md (1)
15-15: Deduplicate and convert URL to Markdown link to satisfy markdownlint MD034.The phrase "Full Changelog" is duplicated and the bare URL triggers MD034. Fix both in one go.
-**Full Changelog**: **Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 +**Full Changelog**: [v0.34.0...v0.35.0](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0)docs/changelogs/v0.35.0.md (2)
13-13: Fix author handle: use @klinch0 (zero) for consistency.Matches usage elsewhere in repo/PR.
-* [cozystack-api] Configure dynamic API using Custom Resources. (@klinch in https://github.com/cozystack/cozystack/pull/1230) +* [cozystack-api] Configure dynamic API using Custom Resources. (@klinch0 in https://github.com/cozystack/cozystack/pull/1230)
68-68: Fix extra space and ClickHouse casing.Removes a formatting glitch and aligns product name casing.
-* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) +* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete ClickHouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236)
🧹 Nitpick comments (1)
docs/changelogs/v0.35.0.md (1)
77-78: Wrap comparison URLs as Markdown links to address MD034 (bare URLs).Keeps the visible labels while satisfying markdownlint.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0-beta.1 -**Since Last Release**: https://github.com/cozystack/cozystack/compare/v0.35.0-beta.1...v0.35.0 +**Full Changelog**: [v0.34.0...v0.35.0-beta.1](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0-beta.1) +**Since Last Release**: [v0.35.0-beta.1...v0.35.0](https://github.com/cozystack/cozystack/compare/v0.35.0-beta.1...v0.35.0)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (2)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.35.0.md(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/template.md
15-15: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.35.0.md
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
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)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
27-27: Bare URL used
(MD034, no-bare-urls)
27-27: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
29-29: Bare URL used
(MD034, no-bare-urls)
30-30: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
67-67: Bare URL used
(MD034, no-bare-urls)
68-68: Bare URL used
(MD034, no-bare-urls)
69-69: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.35.0.md
[grammar] ~30-~30: There might be a mistake here.
Context: ...c7e256906e1dbb0f957dc3a205fa77a147d419d) * [virtual-machine] Fix a regression with ...
(QB_NEW_EN)
[style] ~34-~34: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
[grammar] ~37-~37: There might be a mistake here.
Context: ... the volume size limit from 100MB to 30,000MB. (@kvaps in https://github.com/cozystac...
(QB_NEW_EN)
[grammar] ~68-~68: There might be a mistake here.
Context: ...ithub.com//pull/1236) * [dx] When running make commands with `...
(QB_NEW_EN)
[grammar] ~77-~77: There might be a mistake here.
Context: ...zystack/compare/v0.34.0...v0.35.0-beta.1 Since Last Release: https://github.c...
(QB_NEW_EN)
3c27f04 to
3144a9c
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (4)
docs/changelogs/v0.35.0.md (4)
31-31: Fix link text to match repository name.The link text shows "cozyvalue-gen" (singular) but should be "cozyvalues-gen" (plural) to match the actual repository name.
33-33: Fix username consistency.The username appears as "@Klinch" but should be "@klinch0" for consistency with other references in this changelog and the repository.
37-37: Use correct product casing: ClickHouse.The product name should use the canonical casing "ClickHouse" instead of "Clickhouse".
88-88: Remove extra space in attribution.There's an extra space after "in" in the attribution line.
🧹 Nitpick comments (1)
docs/changelogs/v0.35.0.md (1)
4-25: Improve narrative sections for better readability.The introductory sections provide good context but have some incomplete sentences and could benefit from minor improvements for clarity and flow.
Consider these improvements for better readability:
-Ever wanted to add your own managed application in Cozystack? -Since v0.35.0, this is possible. -With +Ever wanted to add your own managed application in Cozystack? +Since v0.35.0, this is possible with the new external application reconciler functionality. -This functionality will further improve in the next releases of Cozystack. +This functionality will continue to improve in future releases of Cozystack. -Cozystack now runs natively on Hetzner dedicated servers thanks to the RobotLB support. -RobotLB runs as a component of Cozystack +Cozystack now runs natively on Hetzner dedicated servers thanks to RobotLB support. +RobotLB runs as an integrated component of Cozystack.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (3)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.35.0.md(1 hunks)docs/changelogs/v0.35.1.md(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- docs/changelogs/template.md
🧰 Additional context used
🪛 LanguageTool
docs/changelogs/v0.35.0.md
[grammar] ~19-~19: There might be a mistake here.
Context: ...r all managed applications in Cozystack, including databases, tenant Kubernetes, ...
(QB_NEW_EN)
[grammar] ~30-~30: There might be a mistake here.
Context: ...ithub.com//pull/1271) * [cozystack-api, apps] Add default values...
(QB_NEW_EN)
[style] ~50-~50: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
[grammar] ~55-~55: There might be a mistake here.
Context: ...c7e256906e1dbb0f957dc3a205fa77a147d419d) * [virtual-machine] Fix a regression with ...
(QB_NEW_EN)
[grammar] ~61-~61: There might be a mistake here.
Context: ... the volume size limit from 100MB to 30,000MB. (@kvaps in https://github.com/cozystac...
(QB_NEW_EN)
[grammar] ~88-~88: There might be a mistake here.
Context: ...ithub.com//pull/1236) * [dx] When running make commands with `...
(QB_NEW_EN)
[grammar] ~97-~97: There might be a mistake here.
Context: ...tack/cozystack/compare/v0.34.0...v0.35.0 Since Last Release: https://github.c...
(QB_NEW_EN)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.35.0.md
28-28: Bare URL used
(MD034, no-bare-urls)
29-29: Bare URL used
(MD034, no-bare-urls)
30-30: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
67-67: Bare URL used
(MD034, no-bare-urls)
67-67: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
82-82: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
88-88: Bare URL used
(MD034, no-bare-urls)
89-89: Bare URL used
(MD034, no-bare-urls)
90-90: Bare URL used
(MD034, no-bare-urls)
91-91: Bare URL used
(MD034, no-bare-urls)
92-92: Bare URL used
(MD034, no-bare-urls)
93-93: Bare URL used
(MD034, no-bare-urls)
97-97: Bare URL used
(MD034, no-bare-urls)
98-98: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.35.1.md
8-8: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (3)
docs/changelogs/v0.35.1.md (1)
1-11: Well-structured changelog with clear formatting.The changelog follows a consistent format with proper HTML comment for the release tag, clear section organization, and appropriate attribution. The content is concise and informative.
docs/changelogs/v0.35.0.md (2)
1-3: LGTM! Consistent release tag format.The HTML comment with the release tag follows the same format as the v0.35.1 changelog, maintaining consistency across the changelog files.
26-98: Comprehensive and well-organized changelog content.The changelog is thorough and well-structured with clear categorization of changes. The content covers all major areas including features, security, fixes, dependencies, documentation, and development/testing improvements. Attribution is consistent throughout and the technical details are appropriate for the target audience.
3144a9c to
3b5b131
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (3)
docs/changelogs/v0.35.0.md (3)
70-70: Fix link text: repository name is cozyvalues-gen (plural).Link text and target should both reference cozyvalues-gen. This was previously flagged.
-* [cozystack-api, apps] Add default values, clear type definitions, value constraints and other improvements to the OpenAPI specs and READMEs by migrating to [cozyvalue-gen](https://github.com/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333) +* [cozystack-api, apps] Add default values, clear type definitions, value constraints and other improvements to the OpenAPI specs and READMEs by migrating to [cozyvalues-gen](https://github.com/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333)
72-72: Author handle consistency: use @klinch0.Attribution elsewhere uses @klinch0; align this entry. Previously reported.
-* [cozystack-api] Provide an API for administrators to define custom managed applications alongside existing managed apps. (@klinch in https://github.com/cozystack/cozystack/pull/1230) +* [cozystack-api] Provide an API for administrators to define custom managed applications alongside existing managed apps. (@klinch0 in https://github.com/cozystack/cozystack/pull/1230)
129-129: Remove extra space before URL.Minor formatting issue. Previously reported.
-* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) +* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236)
🧹 Nitpick comments (6)
docs/changelogs/v0.35.1.md (2)
8-8: Wrap PR link to avoid markdownlint MD034 (bare URLs).Convert the raw URL to a Markdown link. This also improves readability.
-* [cozy-lib] Fix malformed retrieval of `cozyConfig` in the cozy-lib template. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1348) +* [cozy-lib] Fix malformed retrieval of `cozyConfig` in the cozy-lib template. (@lllamnyp in [#1348](https://github.com/cozystack/cozystack/pull/1348))
10-10: Format “Full Changelog” as a Markdown link (fixes MD034).This keeps the style consistent and silences the linter.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.35.0...v0.35.1 +**Full Changelog**: [v0.35.0...v0.35.1](https://github.com/cozystack/cozystack/compare/v0.35.0...v0.35.1)docs/changelogs/v0.35.0.md (4)
12-13: Minor wording: “creating” reads better than “making”.Tiny copy edit for clarity.
-Adding an application requires making an application package, similar to the ones included in Cozystack +Adding an application requires creating an application package, similar to the ones included in Cozystack
65-65: Remove trailing whitespace.Tiny whitespace cleanup.
-Learn more in the [ClickHouse configuration reference](https://cozystack.io/docs/applications/clickhouse/#clickhouse-keeper-parameters). +Learn more in the [ClickHouse configuration reference](https://cozystack.io/docs/applications/clickhouse/#clickhouse-keeper-parameters).
138-138: Format “Full Changelog” link to avoid MD034 and improve readability.Conform to Markdown link format.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 +**Full Changelog**: [v0.34.0...v0.35.0](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0)
69-134: Bare URLs across the changelog trigger MD034; choose a consistent policy.There are many bare PR/commit URLs in lists (tool flagged MD034 in many lines). Options:
- Keep the style (acceptable for changelogs) and disable MD034 for this file/path, or
- Convert bare URLs to Markdown links like #1234/[commit], at least for new entries going forward.
If you prefer disabling the rule for this file, add at the top:
<!-- markdownlint-disable MD034 -->Or, standardize entries to:
(@author in [#1234](...))Do you want me to mass-convert the entries or add a lint-disable header for changelogs?
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (3)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.35.0.md(1 hunks)docs/changelogs/v0.35.1.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/changelogs/template.md
🧰 Additional context used
🪛 LanguageTool
docs/changelogs/v0.35.0.md
[style] ~12-~12: ‘making an application’ might be wordy. Consider a shorter alternative.
Context: ...ck API. Adding an application requires making an application package, similar to the ones included i...
(EN_WORDINESS_PREMIUM_MAKING_AN_APPLICATION)
[grammar] ~14-~14: There might be a mistake here.
Context: ...called CozystackResourceDefinition and a corresponding controller (reconciler) ...
(QB_NEW_EN)
[grammar] ~17-~17: There might be a mistake here.
Context: ...ozystack.io/docs/applications/external/) and an example at [github.com/cozystack/...
(QB_NEW_EN)
[grammar] ~42-~42: There might be a mistake here.
Context: ...s not supported on most cloud providers. For example, Hetzner provides its own Ro...
(QB_NEW_EN)
[grammar] ~45-~45: There might be a mistake here.
Context: ...tack.io/docs/install/providers/hetzner/) to learn more and deploy your own Cozyst...
(QB_NEW_EN)
[grammar] ~51-~51: There might be a mistake here.
Context: ...perations/stretched/seaweedfs-multidc/), you can connect your primary Cozystack c...
(QB_NEW_EN)
[grammar] ~54-~54: There might be a mistake here.
Context: ...configure the SeaweedFS application with a list of IP addresses or CIDR ranges th...
(QB_NEW_EN)
[grammar] ~69-~69: There might be a mistake here.
Context: ...ithub.com//pull/1313) * [cozystack-api, apps] Add default values...
(QB_NEW_EN)
[style] ~91-~91: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
[grammar] ~96-~96: There might be a mistake here.
Context: ...c7e256906e1dbb0f957dc3a205fa77a147d419d) * [virtual-machine] Fix a regression with ...
(QB_NEW_EN)
[grammar] ~102-~102: There might be a mistake here.
Context: ... the volume size limit from 100MB to 30,000MB. (@kvaps in https://github.com/cozystac...
(QB_NEW_EN)
[grammar] ~129-~129: There might be a mistake here.
Context: ...ithub.com//pull/1236) * [dx] When running make commands with `...
(QB_NEW_EN)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.35.0.md
69-69: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
82-82: Bare URL used
(MD034, no-bare-urls)
83-83: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
88-88: Bare URL used
(MD034, no-bare-urls)
89-89: Bare URL used
(MD034, no-bare-urls)
89-89: Bare URL used
(MD034, no-bare-urls)
90-90: Bare URL used
(MD034, no-bare-urls)
91-91: Bare URL used
(MD034, no-bare-urls)
91-91: Bare URL used
(MD034, no-bare-urls)
92-92: Bare URL used
(MD034, no-bare-urls)
93-93: Bare URL used
(MD034, no-bare-urls)
94-94: Bare URL used
(MD034, no-bare-urls)
95-95: Bare URL used
(MD034, no-bare-urls)
96-96: Bare URL used
(MD034, no-bare-urls)
97-97: Bare URL used
(MD034, no-bare-urls)
98-98: Bare URL used
(MD034, no-bare-urls)
98-98: Bare URL used
(MD034, no-bare-urls)
99-99: Bare URL used
(MD034, no-bare-urls)
100-100: Bare URL used
(MD034, no-bare-urls)
101-101: Bare URL used
(MD034, no-bare-urls)
101-101: Bare URL used
(MD034, no-bare-urls)
102-102: Bare URL used
(MD034, no-bare-urls)
103-103: Bare URL used
(MD034, no-bare-urls)
108-108: Bare URL used
(MD034, no-bare-urls)
108-108: Bare URL used
(MD034, no-bare-urls)
113-113: Bare URL used
(MD034, no-bare-urls)
114-114: Bare URL used
(MD034, no-bare-urls)
115-115: Bare URL used
(MD034, no-bare-urls)
116-116: Bare URL used
(MD034, no-bare-urls)
117-117: Bare URL used
(MD034, no-bare-urls)
117-117: Bare URL used
(MD034, no-bare-urls)
118-118: Bare URL used
(MD034, no-bare-urls)
119-119: Bare URL used
(MD034, no-bare-urls)
120-120: Bare URL used
(MD034, no-bare-urls)
121-121: Bare URL used
(MD034, no-bare-urls)
122-122: Bare URL used
(MD034, no-bare-urls)
123-123: Bare URL used
(MD034, no-bare-urls)
128-128: Bare URL used
(MD034, no-bare-urls)
129-129: Bare URL used
(MD034, no-bare-urls)
130-130: Bare URL used
(MD034, no-bare-urls)
131-131: Bare URL used
(MD034, no-bare-urls)
132-132: Bare URL used
(MD034, no-bare-urls)
133-133: Bare URL used
(MD034, no-bare-urls)
134-134: Bare URL used
(MD034, no-bare-urls)
138-138: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.35.1.md
8-8: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (3)
docs/changelogs/v0.35.1.md (2)
2-4: Nice touch: release tag URL captured for traceability.Keeping the GitHub release URL in an HTML comment at the top is helpful and non-intrusive.
1-6: Consider adding a release heading (version and date).If your changelog convention uses headers per release, prepend a heading like “v0.35.1 — YYYY-MM-DD” for scannability. If the project intentionally omits headings, ignore.
Do you want me to align this file with your existing changelog template by inserting a version/date heading at the top?
docs/changelogs/v0.35.0.md (1)
1-4: Keeping the release tag URL in an HTML comment is good practice.This mirrors the pattern in v0.35.1 and helps back-reference releases without affecting rendered content.
3b5b131 to
22e6223
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (4)
docs/changelogs/template.md (1)
18-20: Deduplicate “Full Changelog” and fix markdownlint MD034 (no-bare-urls).The phrase is repeated and the bare URL triggers the linter. Convert to a single labeled markdown link.
--- -**Full Changelog**: **Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 +**Full Changelog**: [v0.34.0...v0.35.0](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0)Optional: since this is a template, consider using placeholders to avoid stale version numbers (e.g., vX.Y.Z...vA.B.C).
docs/changelogs/v0.35.0.md (3)
72-72: Unify author handle: use @klinch0 for consistency.Other entries use @klinch0; update this one too.
-* [cozystack-api] Provide an API for administrators to define custom managed applications alongside existing managed apps. (@klinch in https://github.com/cozystack/cozystack/pull/1230) +* [cozystack-api] Provide an API for administrators to define custom managed applications alongside existing managed apps. (@klinch0 in https://github.com/cozystack/cozystack/pull/1230)
129-129: Remove extra space before the URL.There are two spaces between “in” and the link.
-* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) +* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236)
69-71: Fix link text to match repository name: cozyvalues-gen (plural).The link text says “cozyvalue-gen” but the repo is “cozyvalues-gen”. Keep the URL as-is; just fix the label.
-* [cozystack-api, apps] Add default values, clear type definitions, value constraints and other improvements to the OpenAPI specs and READMEs by migrating to [cozyvalue-gen](https://github.com/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333) +* [cozystack-api, apps] Add default values, clear type definitions, value constraints and other improvements to the OpenAPI specs and READMEs by migrating to [cozyvalues-gen](https://github.com/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333)
🧹 Nitpick comments (2)
docs/changelogs/v0.35.0.md (2)
12-13: Tighten wording: “making an application package” → “creating an application package”.Minor style tweak for concision and tone.
-Adding an application requires making an application package, similar to the ones included in Cozystack +Adding an application requires creating an application package, similar to the ones included in Cozystack
136-139: Fix markdownlint MD034 (no-bare-urls) for “Full Changelog”.Convert the bare URL to a labeled markdown link. This matches the template change.
--- -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 +**Full Changelog**: [v0.34.0...v0.35.0](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (3)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.35.0.md(1 hunks)docs/changelogs/v0.35.1.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/changelogs/v0.35.1.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/template.md
20-20: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.35.0.md
[style] ~12-~12: ‘making an application’ might be wordy. Consider a shorter alternative.
Context: ...ck API. Adding an application requires making an application package, similar to the ones included i...
(EN_WORDINESS_PREMIUM_MAKING_AN_APPLICATION)
[grammar] ~14-~14: There might be a mistake here.
Context: ...called CozystackResourceDefinition and a corresponding controller (reconciler) ...
(QB_NEW_EN)
[grammar] ~17-~17: There might be a mistake here.
Context: ...ozystack.io/docs/applications/external/) and an example at [github.com/cozystack/...
(QB_NEW_EN)
[grammar] ~42-~42: There might be a mistake here.
Context: ...s not supported on most cloud providers. For example, Hetzner provides its own Ro...
(QB_NEW_EN)
[grammar] ~45-~45: There might be a mistake here.
Context: ...tack.io/docs/install/providers/hetzner/) to learn more and deploy your own Cozyst...
(QB_NEW_EN)
[grammar] ~51-~51: There might be a mistake here.
Context: ...perations/stretched/seaweedfs-multidc/), you can connect your primary Cozystack c...
(QB_NEW_EN)
[grammar] ~54-~54: There might be a mistake here.
Context: ...configure the SeaweedFS application with a list of IP addresses or CIDR ranges th...
(QB_NEW_EN)
[grammar] ~69-~69: There might be a mistake here.
Context: ...ithub.com//pull/1313) * [cozystack-api, apps] Add default values...
(QB_NEW_EN)
[style] ~91-~91: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
[grammar] ~96-~96: There might be a mistake here.
Context: ...c7e256906e1dbb0f957dc3a205fa77a147d419d) * [virtual-machine] Fix a regression with ...
(QB_NEW_EN)
[grammar] ~102-~102: There might be a mistake here.
Context: ... the volume size limit from 100MB to 30,000MB. (@kvaps in https://github.com/cozystac...
(QB_NEW_EN)
[grammar] ~129-~129: There might be a mistake here.
Context: ...ithub.com//pull/1236) * [dx] When running make commands with `...
(QB_NEW_EN)
22e6223 to
d089ef5
Compare
Signed-off-by: Nick Volynkin <nick.volynkin@gmail.com>
d089ef5 to
c10f624
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (4)
docs/changelogs/template.md (1)
20-20: Deduplicate “Full Changelog” label and avoid bare URL.The phrase is repeated twice and the bare URL triggers MD034. This was flagged previously; applying it in the template prevents propagation to new releases.
-**Full Changelog**: **Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 +**Full Changelog**: [v0.34.0...v0.35.0](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0)docs/changelogs/v0.35.0.md (3)
72-72: Use the correct GitHub handle: @klinch0.This was raised earlier but still appears as “@Klinch” here.
-* [cozystack-api] Provide an API for administrators to define custom managed applications alongside existing managed apps. (@klinch in https://github.com/cozystack/cozystack/pull/1230) +* [cozystack-api] Provide an API for administrators to define custom managed applications alongside existing managed apps. (@klinch0 in https://github.com/cozystack/cozystack/pull/1230)
129-129: Use product’s canonical casing and remove extra space.“ClickHouse” is the correct brand casing, and there’s an extra space before the URL.
-* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete Clickhouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236) +* [tests, ci] Simplify test discovery logic; run two k8s tests as separate jobs; delete ClickHouse application after a successful test. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1236)
70-70: Fix link text and target for cozyvalues-gen (typo and incorrect URL).Displayed name should be “cozyvalues-gen” (plural) and the URL should point to the org repo, not “cozystack/cozystack/cozyvalues-gen”.
-* [cozystack-api, apps] Add default values, clear type definitions, value constraints and other improvements to the OpenAPI specs and READMEs by migrating to [cozyvalue-gen](https://github.com/cozystack/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333) +* [cozystack-api, apps] Add default values, clear type definitions, value constraints, and other improvements to the OpenAPI specs and READMEs by migrating to [cozyvalues-gen](https://github.com/cozystack/cozyvalues-gen). (@kvaps and @NickVolynkin in https://github.com/cozystack/cozystack/pull/1216, https://github.com/cozystack/cozystack/pull/1314, https://github.com/cozystack/cozystack/pull/1316, https://github.com/cozystack/cozystack/pull/1321, and https://github.com/cozystack/cozystack/pull/1333)
🧹 Nitpick comments (6)
docs/changelogs/v0.35.2.md (3)
8-8: Use canonical casing for lldpd.Upstream project styles the daemon name as “lldpd” (lowercase). Keep “LLDP” for the protocol acronym, but prefer “lldpd” for the component name for accuracy/consistency.
-* [talos] Add LLDPD (`ghcr.io/siderolabs/lldpd`) as a built-in system extension, enabling LLDP-based neighbor discovery out of the box. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1351 and https://github.com/cozystack/cozystack/pull/1360) +* [talos] Add lldpd (`ghcr.io/siderolabs/lldpd`) as a built-in system extension, enabling LLDP-based neighbor discovery out of the box. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1351 and https://github.com/cozystack/cozystack/pull/1360)
13-13: Tighten grammar and clarity for the SeaweedFS fix.“In which … could be moved, causing … failures” reads better and avoids the conditional “would be moved.”
-* [seaweedfs] Fix a problem where S3 gateway would be moved to an external pod, resulting in authentication failure. (@kvaps in https://github.com/cozystack/cozystack/pull/1361) +* [seaweedfs] Fix an issue in which the S3 gateway could be moved to an external pod, causing authentication failures. (@kvaps in https://github.com/cozystack/cozystack/pull/1361)
22-22: Avoid bare URL for the Full Changelog (markdownlint MD034).Switch to an inline link to satisfy MD034 and match the style used elsewhere.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.35.1...v0.35.2 +**Full Changelog**: [v0.35.1...v0.35.2](https://github.com/cozystack/cozystack/compare/v0.35.1...v0.35.2)docs/changelogs/v0.35.0.md (3)
12-15: Streamline wording: “making an application package.”Minor style tweak to reduce wordiness and improve flow.
-Adding an application requires making an application package, similar to the ones included in Cozystack +Adding an application requires creating a package, similar to the ones included in Cozystack under [`packages/apps`](https://github.com/cozystack/cozystack/tree/main/packages/apps).
91-91: Clarify phrasing: “Disable VPA for VPA …”.Redundant/ambiguous wording. Recommend simplifying.
-* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature, and it was not designed to work in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301 and https://github.com/cozystack/cozystack/pull/1318) +* [kubernetes] Disable VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature; it was enabled by mistake. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301 and https://github.com/cozystack/cozystack/pull/1318)
138-138: Avoid bare URL for the Full Changelog (markdownlint MD034).Make the “Full Changelog” a proper link for consistency and to satisfy linting.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0 +**Full Changelog**: [v0.34.0...v0.35.0](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.35.0)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (4)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.35.0.md(1 hunks)docs/changelogs/v0.35.1.md(1 hunks)docs/changelogs/v0.35.2.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/changelogs/v0.35.1.md
🧰 Additional context used
🪛 LanguageTool
docs/changelogs/v0.35.0.md
[style] ~12-~12: ‘making an application’ might be wordy. Consider a shorter alternative.
Context: ...ck API. Adding an application requires making an application package, similar to the ones included i...
(EN_WORDINESS_PREMIUM_MAKING_AN_APPLICATION)
[grammar] ~14-~14: There might be a mistake here.
Context: ...called CozystackResourceDefinition and a corresponding controller (reconciler) ...
(QB_NEW_EN)
[grammar] ~17-~17: There might be a mistake here.
Context: ...ozystack.io/docs/applications/external/) and an example at [github.com/cozystack/...
(QB_NEW_EN)
[grammar] ~42-~42: There might be a mistake here.
Context: ...s not supported on most cloud providers. For example, Hetzner provides its own Ro...
(QB_NEW_EN)
[grammar] ~45-~45: There might be a mistake here.
Context: ...tack.io/docs/install/providers/hetzner/) to learn more and deploy your own Cozyst...
(QB_NEW_EN)
[grammar] ~51-~51: There might be a mistake here.
Context: ...perations/stretched/seaweedfs-multidc/), you can connect your primary Cozystack c...
(QB_NEW_EN)
[grammar] ~54-~54: There might be a mistake here.
Context: ...configure the SeaweedFS application with a list of IP addresses or CIDR ranges th...
(QB_NEW_EN)
[grammar] ~69-~69: There might be a mistake here.
Context: ...ithub.com//pull/1313) * [cozystack-api, apps] Add default values...
(QB_NEW_EN)
[style] ~91-~91: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
[grammar] ~96-~96: There might be a mistake here.
Context: ...c7e256906e1dbb0f957dc3a205fa77a147d419d) * [virtual-machine] Fix a regression with ...
(QB_NEW_EN)
[grammar] ~102-~102: There might be a mistake here.
Context: ... the volume size limit from 100MB to 30,000MB. (@kvaps in https://github.com/cozystac...
(QB_NEW_EN)
[grammar] ~129-~129: There might be a mistake here.
Context: ...ithub.com//pull/1236) * [dx] When running make commands with `...
(QB_NEW_EN)
docs/changelogs/v0.35.2.md
[grammar] ~12-~12: There might be a mistake here.
Context: ...ithub.com//pull/1353) * [seaweedfs] Fix a problem where S3 gatew...
(QB_NEW_EN)
[grammar] ~13-~13: There might be a mistake here.
Context: .../pull/1353) * [seaweedfs] Fix a problem where S3 gateway would be moved to an externa...
(QB_NEW_EN)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.35.0.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/template.md
20-20: Bare URL used
(MD034, no-bare-urls)
Summary by CodeRabbit