Skip to content

docs: SK-3261 Update flowvault install snippets to 1.3.0 - #456

Open
promptless[bot] wants to merge 2 commits into
mainfrom
promptless/sk-3261-flowvault-readme-1.3.0
Open

promptless[bot] wants to merge 2 commits into
mainfrom
promptless/sk-3261-flowvault-readme-1.3.0

Conversation

@promptless

@promptless promptless Bot commented Oct 29, 2026

Copy link
Copy Markdown

Open in Promptless

Why

  • SK-3261 Bump flowvault version to 1.3.0 #455 sets flowvault/pom.xml to 1.3.0 for the flowvault/v1.3.0 public release.
  • The Gradle and Maven install snippets in flowvault/README.md still pin com.skyflow:skyflow-flowvault-java:1.2.0. The README says these snippets pin the latest release, so they need to match it.

Goal

  • Update both install snippets to 1.3.0.
  • Merge this after 1.3.0 is on Maven Central, or merge it together with the release. You'll know it's published when 1.3.0 shows up at https://central.sonatype.com/artifact/com.skyflow/skyflow-flowvault-java. Until then, readers who copy the snippet get a dependency resolution error.
  • Non-goal: other content changes in the README.

Testing

  • Documentation-only change. I checked that no other file in the repository pins the FlowVault Java version.

Trigger Events

@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 92.38%. Comparing base (062d679) to head (0dfe62e).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #456   +/-   ##
=======================================
  Coverage   92.38%   92.38%           
=======================================
  Files         249      249           
  Lines        8799     8799           
  Branches     1268     1268           
=======================================
  Hits         8129     8129           
  Misses        446      446           
  Partials      224      224           
Flag Coverage Δ
common 92.65% <0.00%> (ø)
flowvault 91.54% <0.00%> (ø)
skyvault 93.77% <0.00%> (ø)

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

Components Coverage Δ
Module: common 92.65% <0.00%> (ø)
Module: skyvault 93.77% <0.00%> (ø)
Module: flowvault 91.54% <0.00%> (ø)
Service Account 87.76% <0.00%> (ø)
Vault Data 92.89% <0.00%> (ø)
Vault Tokens 99.08% <0.00%> (ø)
Vault Connection 100.00% <0.00%> (ø)
Vault Controller 82.78% <0.00%> (ø)
Detect 99.42% <0.00%> (ø)
Audit 100.00% <0.00%> (ø)
BIN Lookup 100.00% <0.00%> (ø)
Config 96.26% <0.00%> (ø)
Utils 91.72% <0.00%> (ø)
Errors 100.00% <0.00%> (ø)
Enums 99.02% <0.00%> (ø)
Logs 100.00% <0.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 062d679...0dfe62e. 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 marked this pull request as ready for review October 29, 2026 13:09
…owvault-readme-1.3.0

# Conflicts:
#	flowvault/README.md
@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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants