Closed addisonbeck closed 6 days ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 29.46%. Comparing base (
5ce4e8f
) to head (ce86636
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One โ Scan Summary & Details โ e4549610-d300-4600-9260-7607cee0e437
Severity | Issue | Source File / Package | Checkmarx Insight |
---|---|---|---|
![]() |
Unpinned Actions Full Length Commit SHA | /build-cli.yml: 402 | Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps... |
Severity | Issue | Source File / Package |
---|---|---|
![]() |
Unpinned Actions Full Length Commit SHA | /build-cli.yml: 406 |
๐๏ธ Tracking
https://bitwarden.atlassian.net/browse/AC-2812
๐ Objective
We've decided to cut the
bit
prefix from the new CLI artifacts so that instead of publishingbw-bit-{OS}.zip
andbw-oss-{OS}.zip
we just publishbw-{OS}.zip
andbw-oss-{OS}.zip
.This PR achieves this by updating the build job only add an artifact prefix to the OSS build, and adjusting the release job to download the correct artifacts.
๐ธ Screenshots
โฐ 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