Skip to content

chore: bump version, plus small change to changelog tooling (#290) (cherry-pick)#291

Merged
lockwobr merged 1 commit into
release/v0.17.xfrom
release-bump-chart
Jun 26, 2026
Merged

chore: bump version, plus small change to changelog tooling (#290) (cherry-pick)#291
lockwobr merged 1 commit into
release/v0.17.xfrom
release-bump-chart

Conversation

@lockwobr

Copy link
Copy Markdown
Collaborator

update docs to reflect this tooling change to match process

Description

Checklist

  • I am familiar with the Contributing Guidelines.
  • My commits are signed off (git commit -s) per the DCO.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

update docs to reflect this tooling change to match process
@lockwobr lockwobr self-assigned this Jun 26, 2026
@lockwobr lockwobr requested a review from a team June 26, 2026 19:34
@lockwobr lockwobr enabled auto-merge (squash) June 26, 2026 19:34
@lockwobr lockwobr merged commit 3269cd9 into release/v0.17.x Jun 26, 2026
26 of 27 checks passed
@lockwobr lockwobr deleted the release-bump-chart branch June 26, 2026 19:35
@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: d7488614-c25f-42e0-881b-834abfb921a0

📥 Commits

Reviewing files that changed from the base of the PR and between 30cebda and ace80d5.

📒 Files selected for processing (5)
  • README.md
  • chart/CHANGELOG.md
  • chart/Chart.yaml
  • docs/release-process.md
  • scripts/gen-changelog.sh

📝 Walkthrough

Walkthrough

Updates the chart release to v0.17.1 by bumping the chart version, changing the README install command, and adding the new changelog entry. It also adds release-process documentation for patch versus minor/major changelog cuts and updates scripts/gen-changelog.sh to source patch-release sections from the matching release branch, with missing-branch and empty-range handling.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • NVIDIA/nodewright#290: Covers the same chart v0.17.1 bump and patch-release changelog generation changes.

Suggested labels

doc, component/chart, component/ci

Suggested reviewers

  • ayuskauskas
  • rice-riley
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release-bump-chart

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added doc Documentation change (PR path label; doc issues use the Documentation type) component/operator Skyhook operator (controller-manager) component/chart Helm chart component/ci CI workflows, GitHub Actions, and repo tooling labels Jun 26, 2026
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 28260838168

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Warning

No base build found for commit 30cebda on release/v0.17.x.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 81.925%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 9914
Covered Lines: 8122
Line Coverage: 81.92%
Coverage Strength: 9.31 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/chart Helm chart component/ci CI workflows, GitHub Actions, and repo tooling component/operator Skyhook operator (controller-manager) doc Documentation change (PR path label; doc issues use the Documentation type)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants