Skip to content
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
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
20 changes: 20 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
# Please see the documentation for all configuration options:
# https://docs.github.com/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file

version: 2
updates:
- package-ecosystem: github-actions
directory: /
schedule:
interval: "quarterly"
groups:
actions:
patterns:
- "action.yaml"
- "ci.yaml"
update-types:
- patch
- minor
- major
Comment thread
Zeitsperre marked this conversation as resolved.
Outdated
cooldown:
default-days: 7
Comment thread
Zeitsperre marked this conversation as resolved.
Outdated
6 changes: 3 additions & 3 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,13 +11,13 @@ jobs:
runs-on: [ubuntu-latest]
strategy:
matrix:
python-version: ["3.10", "3.11", "3.12", "3.13"]
python-version: ["3.10", "3.11", "3.12", "3.13", "3.14"]

steps:
- name: clone the repository
uses: actions/checkout@v3
uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.2
- name: setup python
uses: actions/setup-python@v4
uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
with:
python-version: ${{ matrix.python-version }}
- name: upgrade pip
Expand Down
10 changes: 5 additions & 5 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,24 +3,24 @@ ci:

repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v5.0.0
rev: v6.0.0
hooks:
- id: trailing-whitespace
- id: end-of-file-fixer

- repo: https://github.com/psf/black-pre-commit-mirror
rev: 25.1.0
rev: 26.1.0
hooks:
- id: black

- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.12.2
rev: v0.15.0
hooks:
- id: ruff
- id: ruff-check
args: ["--fix", "--show-fixes"]

- repo: https://github.com/rbubley/mirrors-prettier
rev: v3.6.2
rev: v3.8.1
hooks:
- id: prettier
args: ["--cache-location=.prettier_cache/cache"]
Expand Down
2 changes: 1 addition & 1 deletion action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ runs:
run: |
python $GITHUB_ACTION_PATH/parse_logs.py ${{ inputs.log-path }}
Comment on lines 43 to 47
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd probably use uv run for this (in a new PR). Am I correct in assuming that what you're looking for is a lock file?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, precisely. I'm not using uv yet, but that's what I had in mind using something like:

python -m pip install --require-hashes -r CI/requirements.txt

Copy link
Copy Markdown
Contributor

@keewis keewis Feb 3, 2026

Choose a reason for hiding this comment

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

the only reason I used pip when writing this action is because that was the one I was most familiar with at the time. Considering that using uv would allow us to use a fully isolated env for this action and thus avoid clashing with the activated environment, I believe that would actually be a welcome side-effect.

- name: create the issue
uses: actions/github-script@v7
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v8.0.0
with:
github-token: ${{ github.token }}
script: |
Expand Down
12 changes: 4 additions & 8 deletions parse_logs.py
Original file line number Diff line number Diff line change
Expand Up @@ -149,17 +149,15 @@ def format_summary(report):


def format_report(summaries, py_version):
template = textwrap.dedent(
"""\
template = textwrap.dedent("""\
<details><summary>Python {py_version} Test Summary</summary>

```
{summaries}
```

</details>
"""
)
""")
# can't use f-strings because that would format *before* the dedenting
message = template.format(summaries="\n".join(summaries), py_version=py_version)
return message
Expand Down Expand Up @@ -228,8 +226,7 @@ def compressed_report(reports, max_chars, **formatter_kwargs):


def format_collection_error(error, **formatter_kwargs):
return textwrap.dedent(
"""\
return textwrap.dedent("""\
<details><summary>Python {py_version} Test Summary</summary>

{name} failed:
Expand All @@ -238,8 +235,7 @@ def format_collection_error(error, **formatter_kwargs):
```

</details>
"""
).format(py_version=py_version, name=error.name, traceback=error.repr_)
""").format(py_version=py_version, name=error.name, traceback=error.repr_)


if __name__ == "__main__":
Expand Down