-
Notifications
You must be signed in to change notification settings - Fork 20
[bre-1104] update npm publish workflow to add latest tag #437
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Great job! No new security vulnerabilities introduced in this pull request |
|
Codecov Reportโ
All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #437 +/- ##
=======================================
Coverage 76.93% 76.93%
=======================================
Files 270 270
Lines 25608 25608
=======================================
Hits 19702 19702
Misses 5906 5906 โ View full report in Codecov by Sentry. ๐ New features to boost your workflow:
|
โฆ npm publish workflow to add latest tag (bitwarden/sdk-internal#437)
## ๐๏ธ Tracking [bre-1104](https://bitwarden.atlassian.net/browse/bre-1104) ## ๐ Objective related to #415 and #437 example failure after moving to oidc npm publish -> https://github.com/bitwarden/sdk-internal/actions/runs/17654697060/job/50174358624 it is searching for provenance information in the package.json file but the repository url is not defined there ## โฐ Reminders before review - Contributor guidelines followed - All formatters and local linters executed and passed - Written new unit and / or integration tests where applicable - Protected functional changes with optionality (feature flags) - Used internationalization (i18n) for all UI strings - CI builds passed - Communicated to DevOps any deployment requirements - Updated any necessary documentation (Confluence, contributing docs) or informed the documentation team ## ๐ฆฎ Reviewer guidelines <!-- Suggested interactions but feel free to use (or not) as you desire! --> - ๐ (`:+1:`) or similar for great changes - ๐ (`:memo:`) or โน๏ธ (`:information_source:`) for notes or general info - โ (`:question:`) for questions - ๐ค (`:thinking:`) or ๐ญ (`:thought_balloon:`) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion - ๐จ (`:art:`) for suggestions / improvements - โ (`:x:`) orโ ๏ธ (`:warning:`) for more significant problems or concerns needing attention - ๐ฑ (`:seedling:`) or โป๏ธ (`:recycle:`) for future improvements or indications of technical debt - โ (`:pick:`) for minor or nitpick changes
๐๏ธ Tracking
bre-1104
๐ Objective
after merging #415 , based off moving to npm version 11 a
--tag
must be provided for all pre-releases in NPM. our releases are considered pre-release bc of the dashes and suffixes in the naming convention.failure example: https://github.com/bitwarden/sdk-internal/actions/runs/17653063030/job/50172366269
โฐ Reminders before review
๐ฆฎ Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or โน๏ธ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or ๐ญ (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or:warning:
) for more significant problems or concerns needing attention:seedling:
) or โป๏ธ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes