Skip to content

ci: publish combined SDK with GitHub Actions OIDC - #1693

Closed
congminh1254 wants to merge 2 commits into
combined-sdkfrom
ci/npm-oidc-combined-sdk
Closed

congminh1254 wants to merge 2 commits into
combined-sdkfrom
ci/npm-oidc-combined-sdk

Conversation

@congminh1254

Copy link
Copy Markdown
Member

Summary

  • add .github/workflows/publish.yml to the combined-sdk release line
  • publish version tags through npm Trusted Publishing with GitHub OIDC
  • validate the package version matches the release tag before publishing

This ensures v4 releases created from combined-sdk trigger the npm publish workflow.

box-sdk-build and others added 2 commits November 9, 2026 19:31
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedgithub/​actions/​setup-node@​49933ea5288caeca8642d1e84afbd3f7d68200208610010010080

View full report

@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 42675586770

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 →

Coverage remained the same at 15.975%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 1 of 1 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 45424
Covered Lines: 11702
Line Coverage: 25.76%
Relevant Branches: 35539
Covered Branches: 1232
Branch Coverage: 3.47%
Branches in Coverage %: Yes
Coverage Strength: 105.05 hits per line

💛 - Coveralls

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.

4 participants