Skip to content
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

ci(digest): Add digest pinning #149

Merged
merged 1 commit into from
Oct 20, 2023
Merged

ci(digest): Add digest pinning #149

merged 1 commit into from
Oct 20, 2023

Conversation

merkata
Copy link
Contributor

@merkata merkata commented Oct 18, 2023

Applicable spec:

Overview

Adds digest pinning to rockfiles.

Rationale

Adds digest pinning to rockfiles.

Juju Events Changes

n/a

Module Changes

n/a

Library Changes

n/a

Checklist

@merkata merkata added the trivial This is a trivial PR label Oct 18, 2023
@merkata merkata requested a review from a team as a code owner October 18, 2023 21:28
@github-actions
Copy link
Contributor

Test coverage for 0783aeb

Name                Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------
src/charm.py          521     37    168     29    90%   197-200, 551, 582, 628, 663-664, 715-722, 727, 829->834, 833, 835, 840-841, 901, 919, 926, 1018, 1027, 1039, 1060, 1069, 1088, 1092, 1121, 1174, 1306, 1328, 1335->1337, 1380->exit, 1392, 1424, 1433-1434
src/cos.py             15      0      0      0   100%
src/exceptions.py      17      1      2      1    89%   41
src/types_.py          15      0      0      0   100%
---------------------------------------------------------------
TOTAL                 568     38    170     30    90%

Static code analysis report

Run started:2023-10-18 21:45:17.965783

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 4250
  Total lines skipped (#nosec): 1
  Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
  Total issues (by severity):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
  Total issues (by confidence):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
Files skipped (0):

@arturo-seijas arturo-seijas merged commit eb1af43 into main Oct 20, 2023
25 checks passed
@arturo-seijas arturo-seijas deleted the feat/digest-pinning branch October 20, 2023 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants