Skip to content

update release secret names - #216

Merged
isaacbrodsky merged 4 commits into
uber:masterfrom
isaacbrodsky:update-release-secret-names
Oct 15, 2026
Merged

isaacbrodsky merged 4 commits into
uber:masterfrom
isaacbrodsky:update-release-secret-names

Conversation

@isaacbrodsky

@isaacbrodsky isaacbrodsky commented Oct 11, 2026 •

Copy link
Copy Markdown
Collaborator
  • Update release secret names.
  • Release action was incorrectly taking the entire file contents, rather than just the section for this release. Claude suggested this awk invocation to get just the relevant section.
  • Pinned softprops/action-gh-release, including a major version bump.
  • Pinned the release workflow running to uber/h3-java to avoid incorrectly running for a fork, where it would not work.

I haven't removed the old secrets yet. This needs to be tested by running a new release.

@coveralls

coveralls commented Oct 11, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 681

Coverage remained the same at 96.387%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 580
Covered Lines: 567
Line Coverage: 97.76%
Relevant Branches: 195
Covered Branches: 180
Branch Coverage: 92.31%
Branches in Coverage %: Yes
Coverage Strength: 0.98 hits per line

💛 - Coveralls

@isaacbrodsky
isaacbrodsky merged commit 03d9073 into uber:master Oct 15, 2026
69 checks passed
@isaacbrodsky
isaacbrodsky deleted the update-release-secret-names branch October 15, 2026 15:15
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