Skip to content
Merged
Show file tree
Hide file tree
Changes from all 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
17 changes: 0 additions & 17 deletions .github/workflows/make-based-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,23 +13,6 @@ concurrency:
permissions: {}

jobs:
poetry-checks:
# FIXME: Remove when https://github.com/target/make-python-devex/issues/119 is fixed
continue-on-error: true
runs-on: ubuntu-latest
permissions:
pull-requests: write
contents: read
steps:
- name: Checkout code
uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
with:
persist-credentials: false
# This will post a comment on PRs when poetry.lock changes
- name: Diff poetry.lock
uses: nborrmann/diff-poetry-lock@c0afd3666864cec339f03ee5c5cf9cb671780c12


build:
strategy:
matrix:
Expand Down
30 changes: 30 additions & 0 deletions .github/workflows/poetry-checks.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
name: poetry checks

on:
pull_request:
branches: [ "main" ]

concurrency:
group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
cancel-in-progress: true

permissions: {}

jobs:
diff-poetry-lock:
# FIXME: Remove when https://github.com/target/make-python-devex/issues/119 is fixed
continue-on-error: true
runs-on: ubuntu-latest
permissions:
pull-requests: write
issues: write
contents: read
steps:
- name: Checkout code
uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
with:
persist-credentials: false
# This will post a comment on PRs when poetry.lock changes
- name: Diff poetry.lock
uses: colindean/diff-poetry-lock@nl

Check failure

Code scanning / zizmor

unpinned action reference Error

unpinned action reference
Copy link
Contributor

Choose a reason for hiding this comment

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

We'll accept some short-term risk here.


Loading