Skip to content

chore(docs): update contributing guidelines (charmkeeper)#114

Open
erinecon wants to merge 6 commits intomainfrom
charmkeeper/update-contributing-guidelines
Open

chore(docs): update contributing guidelines (charmkeeper)#114
erinecon wants to merge 6 commits intomainfrom
charmkeeper/update-contributing-guidelines

Conversation

@erinecon
Copy link
Copy Markdown
Contributor

This PR updates the contributing guidelines for the Content Cache charm.

Changes

  • Created docs/changelog.md
  • Replaced CONTRIBUTING.md
  • Deleted .github/pull_request_template.yaml
  • Replaced .github/pull_request_template.md

This PR was automatically created by charmkeeper.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@erinecon erinecon marked this pull request as ready for review April 21, 2026 18:33
@erinecon erinecon requested a review from a team as a code owner April 21, 2026 18:33
@erinecon erinecon requested review from alithethird and arturo-seijas and removed request for a team April 21, 2026 18:33
@erinecon erinecon added the documentation Improvements or additions to documentation label Apr 21, 2026
Comment thread CONTRIBUTING.md Outdated
Copy link
Copy Markdown

@alithethird alithethird left a comment

Choose a reason for hiding this comment

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

Nitpick, otherwise LGTM

@github-actions
Copy link
Copy Markdown
Contributor

Test results for commit 30604e4

Test coverage for 30604e4

Name            Stmts   Miss Branch BrPart  Cover   Missing
-----------------------------------------------------------
src/charm.py       47      0      6      0   100%
src/errors.py       1      0      0      0   100%
src/state.py      139      0     32      0   100%
-----------------------------------------------------------
TOTAL             187      0     38      0   100%

Static code analysis report

Run started:2026-04-22 14:46:06.339812+00:00

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 986
  Total lines skipped (#nosec): 0
  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):

@github-actions
Copy link
Copy Markdown
Contributor

Test results for commit 30604e4

Test coverage for 30604e4

Name                   Stmts   Miss Branch BrPart  Cover   Missing
------------------------------------------------------------------
src/certificates.py       21      2      6      2    85%   65-68
src/charm.py             106     12      8      0    89%   64-65, 120, 149, 153-156, 211-213, 223-225
src/errors.py             16      0      0      0   100%
src/nginx_manager.py     199     36     16      4    81%   207, 213-219, 224-226, 229, 277-279, 295-297, 337-342, 417-419, 438-442, 461->463, 543-548
src/state.py             163     13     34      1    92%   154-159, 429-439
src/utilities.py          13      0      0      0   100%
------------------------------------------------------------------
TOTAL                    518     63     64      7    88%

Static code analysis report

Run started:2026-04-22 14:47:26.659996+00:00

Test results:
  No issues identified.

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

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):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Libraries: OK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants