Skip to content
This repository has been archived by the owner on Jun 7, 2024. It is now read-only.

Update charm libraries #29

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Update charm libraries #29

merged 1 commit into from
Jul 11, 2023

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jul 8, 2023

Automated action to fetch latest version of charm libraries. The branch of this PR
will be wiped during the next check. Unless you really know what you're doing, you
most likely don't want to push any commits to this branch.

@github-actions github-actions bot requested a review from a team as a code owner July 8, 2023 01:34
@github-actions github-actions bot force-pushed the chore/auto-libs branch 2 times, most recently from e632298 to 3e34589 Compare July 10, 2023 01:36
@arturo-seijas arturo-seijas reopened this Jul 10, 2023
@gregory-schiano
Copy link
Contributor

We need to pin pydantic before this PR

@github-actions
Copy link
Contributor Author

Test coverage for 3e34589

Name                    Stmts   Miss Branch BrPart  Cover   Missing
-------------------------------------------------------------------
src/charm.py               96     17     12      4    81%   48-50, 85->exit, 99, 117-119, 135-137, 162-165, 211-212, 214-215
src/charm_state.py         82      1     10      1    98%   25
src/charm_types.py          8      0      0      0   100%
src/constants.py            6      0      0      0   100%
src/databases.py           52     16     15      2    70%   65-83, 125, 140-141
src/exceptions.py           8      1      0      0    88%   40
src/flask_app.py           16      0      8      1    96%   48->50
src/observability.py       12      0      0      0   100%
src/secret_storage.py      35      1      6      2    93%   45->exit, 74
src/webserver.py           70      4     12      1    94%   127-133, 151
-------------------------------------------------------------------
TOTAL                     385     40     63     11    88%

Static code analysis report

Run started:2023-07-10 14:54:44.722263

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 1893
  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):

Copy link
Contributor

@gregory-schiano gregory-schiano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gregory-schiano gregory-schiano merged commit 59a6647 into main Jul 11, 2023
@gregory-schiano gregory-schiano deleted the chore/auto-libs branch July 11, 2023 07:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants