-
Notifications
You must be signed in to change notification settings - Fork 11
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
feat: github app authentication and repo standardization #43
Conversation
That should have been run against python 3.10, not python 3.1. 😕 Re-running. UPDATE: Versions needed to be in string format |
3ef62ed
to
9f876be
Compare
- [x] setting minimum python to 3.10 (allows usage of unions on types) - [x] add ability for users to authenticate with GitHub App Installation - [x] standardize repo - [x] Makefile - [x] Linting - [x] same workflows as other OSPO GitHub Actions - [x] add @jmeridth to CODEOWNERS - [x] update .env-example - [x] update README - [x] standardize all workflows (including examples) permissions - [x] contents: read by default - [x] more details in jobs - [x] add tests - [x] coverage to 60% for now, will get above 80% later - [x] pull request template Signed-off-by: jmeridth <jmeridth@gmail.com>
9f876be
to
c79601a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for handling such a massive change and leveling up the repo to fit with our other actions!
FROM python:3.8-slim-buster | ||
#checkov:skip=CKV_DOCKER_2 | ||
#checkov:skip=CKV_DOCKER_3 | ||
FROM python:3.12-slim |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we are dropping support for 3.8, lets do a major version bump and note in the release notes that the breaking change is dropping python 3.8 support.
Co-authored-by: Zack Koppert <zkoppert@github.com>
Co-authored-by: Zack Koppert <zkoppert@github.com>
Co-authored-by: Zack Koppert <zkoppert@github.com>
cm -> context_manager Signed-off-by: jmeridth <jmeridth@gmail.com>
Signed-off-by: jmeridth <jmeridth@gmail.com>
Since we're about to do a major version release, need to update action.yml to v3. Should have been done in #43 - [x] update action.yml to v3 - [x] update dependencies Signed-off-by: jmeridth <jmeridth@gmail.com>
I renamed the workflow from linter.yml to super-linter.yml in #43 Signed-off-by: jmeridth <jmeridth@gmail.com>
The size of this PR is necessary. Not my usual PR size.
No external changes from a user's perspective but a bit of an overhaul/refactor. I'd still say a minor release, not major. Unless we feel setting minimum python to 3.10. We are building the action with 3.12 now but test against 3.10+.
Pull Request
Proposed Changes
Readiness Checklist
Author/Contributor
make lint
and fix any issues that you have introducedmake test
and ensure you have test coverage for the lines you are introducingReviewer
bug
,documentation
,enhancement
,infrastructure
, orbreaking