Skip to content

Prep to Open Source #3

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

Merged
merged 29 commits into from
Dec 5, 2024
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
38aa0f6
Small fixes for CI and a license
crimsonknave Sep 6, 2024
7f6d8ae
Split coverage action into two
crimsonknave Sep 23, 2024
d389102
Mega commit with ci fixes, and converting to use logging dict
crimsonknave Sep 26, 2024
0dd1713
Apparently, it was really mad I had a typo in the readme name
crimsonknave Sep 27, 2024
a3fcf2b
Not sure how that got uncapitalized
crimsonknave Oct 7, 2024
74814bc
Working on smushing mutations
crimsonknave Oct 7, 2024
09235ae
Cleaning up dependencies and fixing a mutation
crimsonknave Oct 7, 2024
9f6db68
Fixing up CI and mutations
crimsonknave Oct 7, 2024
29fedeb
Adding codeql
crimsonknave Oct 7, 2024
3cbe249
yaml >> yml
crimsonknave Oct 7, 2024
b9d11bd
Killing last mutation
crimsonknave Oct 7, 2024
0f95223
Adding/updating OSS documentation
crimsonknave Oct 7, 2024
56adb9b
Removing custom codeql scanning
crimsonknave Oct 9, 2024
f4f5cf2
Too large commit: Add actions plugin and lots of rework to solve some…
crimsonknave Oct 10, 2024
5bdda75
Adding note about merging logging configs
crimsonknave Oct 10, 2024
47ed59c
Removing mutmut from CI as it is not longer working as expected.
crimsonknave Nov 26, 2024
b847919
Minor cleanup
crimsonknave Nov 26, 2024
24a8ae2
Pin coverage action to a sha
crimsonknave Nov 26, 2024
200e0b9
Cleaning up some code scanning issues
crimsonknave Nov 26, 2024
23c94f6
Fixing up workflows
crimsonknave Nov 27, 2024
84a2086
Readme and coverage ci updates
crimsonknave Nov 27, 2024
e251cdb
Getting the coverage comment to work again
crimsonknave Nov 27, 2024
ba69eff
Fixing action yaml
crimsonknave Nov 27, 2024
28ccdc2
More fixing of CI
crimsonknave Nov 27, 2024
8de95a8
Maybe actually fix the coverage ci
crimsonknave Nov 27, 2024
a2f4d6a
Convert runtime annotations to plugin and additional testing for acti…
crimsonknave Dec 3, 2024
d183d62
Cleanup and removing some support for older configurations
crimsonknave Dec 4, 2024
10ea0dd
Fully test actions log message
crimsonknave Dec 4, 2024
6209b3a
More cleanup
crimsonknave Dec 4, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions .github/workflows/coverage.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@

name: Post Coverage Commit

on:
workflow_run:
workflows: ["Pytest"]
types:
- completed

jobs:
coverage:
runs-on: ubuntu-latest
if: github.event.workflow_run.event == "pull_request" && github.event.workflow_run.conclusion == "success"
permissions:
pull-requests: write
contents: write
actions: read
steps:
- uses: actions/checkout@v4

- uses: actions/download-artifact@v4
id: download
with:
pattern: coverage-*
merge-multiple: true
- name: Re-add relative so the action is happy
run: |
echo "[run]" >> .coveragerc
echo "relative_files = true" >> .coveragerc
- name: Python Coverage Comment
uses: py-cov-action/python-coverage-comment-action@v3.28
with:
GITHUB_TOKEN: ${{ github.token }}
MERGE_COVERAGE_FILES: true
GITHUB_PR_RUN_ID: ${{ github.event.workflow_run.id }}
14 changes: 14 additions & 0 deletions .github/workflows/matchers/ruff.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
{
"problemMatcher": [
{
"owner": "ruff",
"pattern": [
{
"regexp": "^(Would reformat): (.+)$",
"message": 1,
"file": 2
}
]
}
]
}
32 changes: 5 additions & 27 deletions .github/workflows/pytest.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,32 +40,10 @@ jobs:
hatch run dev:pytest
env:
COVERAGE_FILE: ".coverage.${{ matrix.os }}.${{ matrix.python-version }}"
- name: Store coverage file
- name: Store Pull Request comment to be posted
uses: actions/upload-artifact@v4
if: steps.coverage_comment.outputs.COMMENT_FILE_WRITTEN == 'true'
with:
name: coverage-${{ matrix.os }}-${{ matrix.python-version }}
path: .coverage.${{ matrix.os }}.${{ matrix.python-version }}

coverage:
runs-on: ubuntu-latest
needs: pytest
permissions:
pull-requests: write
contents: write
steps:
- uses: actions/checkout@v4

- uses: actions/download-artifact@v4
id: download
with:
pattern: coverage-*
merge-multiple: true
- name: Re-add relative so the action is happy
run: |
echo "[run]" >> .coveragerc
echo "relative_files = true" >> .coveragerc
- name: Python Coverage Comment
uses: py-cov-action/python-coverage-comment-action@v3.24
with:
GITHUB_TOKEN: ${{ github.token }}
MERGE_COVERAGE_FILES: true
name: python-coverage-comment-action
path: python-coverage-comment-action.txt
include-hidden-files: true
9 changes: 7 additions & 2 deletions .github/workflows/ruff.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,11 @@ jobs:
run: |
python -m pip install hatch
hatch env create dev
- name: Lint with Ruff
- name: Lint with Ruff (check)
run: |
hatch run dev:ruff --output-format=github .
hatch run dev:ruff check --output-format=github .
- name: Register problem matcher for ruff format
run: echo "::add-matcher::.github/workflows/matchers/ruff.json"
- name: Lint with Ruff (format)
run: |
hatch run dev:ruff format --check .
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ repos:
# Run the formatter.
- id: ruff-format
- repo: https://github.com/RobertCraigie/pyright-python
rev: v1.1.355
rev: v1.1.382
hooks:
- id: pyright
# I don't love having to specify these here, but pre-commit only seems to work
Expand Down
21 changes: 21 additions & 0 deletions LICENSE.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
MIT License

Copyright GitHub, Inc.

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.
Loading
Loading