Skip to content

SK-3261 Bump flowvault version to 1.3.0 - #455

Merged
yaswanth-pula-skyflow merged 1 commit into
mainfrom
SK-3261/flowvault-version-bump-1.3.0
Oct 29, 2026
Merged

yaswanth-pula-skyflow merged 1 commit into
mainfrom
SK-3261/flowvault-version-bump-1.3.0

Conversation

@yaswanth-pula-skyflow

Copy link
Copy Markdown
Collaborator

Why

  • The public release run for flowvault/v1.3.0 (run 40876207285) failed at Commit changes. The push of the [AUTOMATED] Public Release - 1.3.0 commit to main was rejected with GH013: Changes must be made through a pull request.
  • An org-level ruleset, prod-branch-protection (added 2026-10-09, updated 2026-10-28), now requires a PR for every change to the default branch. The release service-account PAT can bypass the repo rulesets, but it is not on this ruleset's bypass list. Every public release before this ruleset existed (most recently 1.2.0 on 2026-09-10) pushed straight to main.
  • The failure happened before Publish package, so nothing was published to Maven Central.

Goal

  • Set flowvault/pom.xml to 1.3.0 on main through a PR. When the release runs again, the bump step leaves the pom unchanged, Commit changes finds nothing staged and exits 0 without pushing (shared-build-and-deploy.yml, "nothing to commit" guard), and the build and deploy proceed.
  • Non-goal: changing the release workflow or the ruleset. The long-term fix (adding a bypass for the release actor, or making version bumps go through a PR) will be handled separately.

Testing

  • Ran scripts/bump_version.sh 1.3.0 "" flowvault locally against this pom. The output is byte-identical to the input, so nothing is staged when the workflow runs.
  • After merge: publish the flowvault/v1.3.0 release targeting main, with the tag on this PR's merge commit. Don't merge other PRs that touch flowvault/pom.xml before the release runs.

🤖 Generated with Claude Code

The public release workflow can no longer push its version-bump commit
to main: the org ruleset prod-branch-protection requires a pull request
and the release PAT is not on its bypass list. With the pom already at
1.3.0, the workflow's commit step finds nothing staged and skips the push.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

✅ Gitleaks Findings: No secrets detected. Safe to proceed!

@github-actions

Copy link
Copy Markdown

Semgrep Findings: Issues with Error level severity are found (Error is Highest severity in Semgrep), Please resolve the issues before merging.

@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.29%. Comparing base (3c8d25a) to head (dea74ed).

Additional details and impacted files
@@            Coverage Diff             @@
##               main     #455    +/-   ##
==========================================
  Coverage     91.29%   91.29%            
- Complexity        0      571   +571     
==========================================
  Files           174      174            
  Lines          7401     7401            
  Branches       1024     1024            
==========================================
  Hits           6757     6757            
  Misses          422      422            
  Partials        222      222            
Flag Coverage Δ
common 90.82% <ø> (ø)
flowvault 87.92% <ø> (ø)
skyvault 94.98% <ø> (ø)
unittests-flowvault 90.42% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Module: common 90.82% <ø> (ø)
Module: skyvault 94.98% <ø> (ø)
Module: flowvault 87.92% <ø> (ø)
Service Account 87.76% <ø> (ø)
Vault Data 90.81% <ø> (ø)
Vault Tokens 99.03% <ø> (ø)
Vault Connection 100.00% <ø> (ø)
Vault Controller 82.75% <ø> (ø)
Detect 100.00% <ø> (ø)
Audit 100.00% <ø> (ø)
BIN Lookup 100.00% <ø> (ø)
Config 96.26% <ø> (ø)
Utils 90.53% <ø> (ø)
Errors 100.00% <ø> (ø)
Enums 100.00% <ø> (ø)
Logs 100.00% <ø> (ø)

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c8d25a...dea74ed. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@promptless

promptless Bot commented Oct 29, 2026

Copy link
Copy Markdown

Promptless documentation updates

@yaswanth-pula-skyflow
yaswanth-pula-skyflow merged commit c50c677 into main Oct 29, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants