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

Create bot_pr_approval.yaml #236

Merged
merged 2 commits into from
Feb 8, 2024
Merged

Create bot_pr_approval.yaml #236

merged 2 commits into from
Feb 8, 2024

Conversation

arturo-seijas
Copy link
Contributor

@arturo-seijas arturo-seijas commented Feb 8, 2024

Applicable spec: N/A

Overview

Add bot to atomatically add one approval for github bot PRs

Rationale

N/A

Module Changes

N/A

Checklist

  • The contributing guide was applied
  • The documentation is generated using src-docs
  • The PR is tagged with appropriate label (urgent, trivial, complex)
  • Changelog has been updated
  • Version has been incremented

@arturo-seijas arturo-seijas requested a review from a team as a code owner February 8, 2024 15:53
Copy link

github-actions bot commented Feb 8, 2024

Test coverage for 415b7c8

Name                      Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------------
src/__init__.py              97      0     42      0   100%
src/action.py               155      0     46      0   100%
src/check.py                 67      0     27      0   100%
src/clients.py               12      0      0      0   100%
src/commit.py                42      0     12      0   100%
src/constants.py             10      0      0      0   100%
src/content.py               50      0     10      0   100%
src/discourse.py            159      0     34      0   100%
src/docs_directory.py        33      0      8      0   100%
src/download.py              23      0      2      0   100%
src/exceptions.py            15      0      0      0   100%
src/index.py                142      0     56      0   100%
src/metadata.py              28      0     12      0   100%
src/migration.py            102      0     33      0   100%
src/navigation_table.py      65      0     20      0   100%
src/reconcile.py            123      0     60      0   100%
src/repository.py           284      0     88      0   100%
src/sort.py                  43      0     26      0   100%
src/types_.py               201      0     54      0   100%
---------------------------------------------------------------------
TOTAL                      1651      0    530      0   100%

Static code analysis report

Working... ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:02
Run started:2024-02-08 16:34:19.347517

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 17623
  Total lines skipped (#nosec): 3
  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 794d1d4 into main Feb 8, 2024
24 checks passed
@arturo-seijas arturo-seijas deleted the add-bot-approvals branch February 8, 2024 16:54
javierdelapuente added a commit that referenced this pull request Apr 4, 2024
* Fix nothing to migrate bug (#231)

* handle case where default branch and discourse are in-line but base is not for the migration case

* Add a formatted representation of the UpdatePageAction dataclass (#230)

* Add a formatted representation of the UpdatePageAction dataclass

* Update changelog

* Add return value to docstring

* Include return type

* Apply lint changes

* Include a string representation of the class, which many tests check for

* fix linting (#234)

* Update python Docker tag to v3.12 (#220)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: arturo-seijas <[email protected]>

* Update dependency more-itertools to >=10.2,<10.3 (#229)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: arturo-seijas <[email protected]>

* Update dependency PyGithub to >=2.2,<2.3 (#233)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: arturo-seijas <[email protected]>
Co-authored-by: Christopher Bartz <[email protected]>

* Update dependency pytest to v8 (#232)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: arturo-seijas <[email protected]>

* Create bot_pr_approval.yaml (#236)

* Removed `discourse-gatekeeper/discourse-ahead-ok` (#235)

* feat(ISD-1567): removed obsolete code and corresponding documentation

* feat(ISD-1498): removed check in test_check for the removed tag

* feat(ISD-1498): updated test parameters to account for removal of discourse ahead tag removal

* feat(ISD-1498): removed tagged argument from tests

* feat(ISD-1498): ensured all parameters return only 2 values in tuple

* feat(ISD-1498): removed ahead tag from constants file, removed mention in integration tests

* feat(ISD-1498): removed ahead tag from check.py

* feat(ISD-1498): rectified erroneous deletion of expected problem in test check

* feat(ISD-1498): adhered to linting requirements by removing unused src.constants imports

* feat(ISD-1498): fixed all linting issues and fixed method signatures.

* feat(ISD-1498): removed test case 7 from test conflicts

* feat(ISD-1498): removed test 7

* feat(ISD-1498): fixed linting issues

* feat(ISD-1498): updated change log with new version details

* feat(ISD-1498): added src-docs

* feat(ISD-1498): fixed comments in run_conflict integration test

* feat(ISD-1498): removed assert associated with removed act

* Prepare discourse-gatekeeper for PAAS app charmer  (#238)


New input parameter `charm-dir`, that defines where to look for `metadata.yaml`, `charmcraft.yaml` or the docs directory.

If `metadata.yaml` does not exist, `charmcraft.yaml` is used instead.

* conflict change race condition resolution stage 1 (#241)

* add check that content has not changed on discourse before updating the content

* New version v0.9.0 (#242)

---------

Co-authored-by: David Andersson <[email protected]>
Co-authored-by: mthaddon <[email protected]>
Co-authored-by: arturo-seijas <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Christopher Bartz <[email protected]>
Co-authored-by: Brendan Bell <[email protected]>
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