-
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Disable CI linting for now because it doesn't work atm.
- Loading branch information
Showing
1 changed file
with
70 additions
and
62 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,63 +1,71 @@ | ||
name: Linting | ||
# Our token isn't provisioned with permission to actually commit back CI fixed | ||
# Run getsentry/action-github-commit@v2.1.0 | ||
#Error: HttpError: Resource not accessible by integration - https://docs.github.com/rest/git/trees#create-a-tree | ||
# at /home/runner/work/_actions/getsentry/action-github-commit/v2.1.0/dist/index.js:7748:21 | ||
# at process.processTicksAndRejections (node:internal/process/task_queues:95:5) | ||
#Error: Resource not accessible by integration - https://docs.github.com/rest/git/trees#create-a-tree | ||
# TODO: Address this later. | ||
|
||
on: | ||
pull_request: | ||
|
||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }} | ||
cancel-in-progress: true | ||
|
||
jobs: | ||
build: | ||
runs-on: ubuntu-latest | ||
permissions: | ||
pull-requests: read | ||
contents: write | ||
statuses: write | ||
|
||
steps: | ||
- uses: actions/checkout@v3 | ||
- name: Get changed files | ||
id: changes | ||
uses: dorny/paths-filter@0bc4621a3135347011ad047f9ecf449bf72ce2bd # v3.0.0 | ||
with: | ||
# Paths to files will be available in `${FILTER_NAME}_files` output variable. | ||
list-files: json | ||
filters: | | ||
all: | ||
- added|modified: '**/*' | ||
# TODO: split out requirements-dev.txt | ||
- uses: getsentry/action-setup-venv@a133e6fd5fa6abd3f590a1c106abda344f5df69f # v2.1.0 | ||
with: | ||
python-version: 3.11.8 | ||
cache-dependency-path: | | ||
requirements.txt | ||
install-cmd: pip install -r requirements-precommit.txt | ||
- name: Setup pre-commit | ||
run: | | ||
pre-commit install | ||
- name: Run pre-commit on PR commits | ||
id: pre-commit_results | ||
continue-on-error: true | ||
run: | | ||
jq '.[]' --raw-output <<< '${{steps.changes.outputs.all_files}}' | | ||
# Run pre-commit to lint and format check files that were changed (but not deleted) compared to master. | ||
xargs pre-commit run --files | ||
- name: Get auth token | ||
id: token | ||
if: ${{ steps.pre-commit_results.outcome == 'failure' }} | ||
uses: getsentry/action-github-app-token@v3.0.0 | ||
with: | ||
app_id: ${{ vars.SENTRY_INTERNAL_APP_ID }} | ||
private_key: ${{ secrets.SENTRY_INTERNAL_APP_PRIVATE_KEY }} | ||
|
||
- name: Apply any pre-commit fixed files | ||
if: ${{ steps.pre-commit_results.outcome == 'failure' }} | ||
# note: this runs "always" or else it's skipped when pre-commit fails | ||
uses: getsentry/action-github-commit@v2.1.0 | ||
with: | ||
github-token: ${{ steps.token.outputs.token }} | ||
message: ':hammer_and_wrench: apply pre-commit fixes' | ||
#name: Linting | ||
# | ||
#on: | ||
# pull_request: | ||
# | ||
#concurrency: | ||
# group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }} | ||
# cancel-in-progress: true | ||
# | ||
#jobs: | ||
# build: | ||
# runs-on: ubuntu-latest | ||
# permissions: | ||
# pull-requests: read | ||
# contents: write | ||
# statuses: write | ||
# | ||
# steps: | ||
# - uses: actions/checkout@v3 | ||
# - name: Get changed files | ||
# id: changes | ||
# uses: dorny/paths-filter@0bc4621a3135347011ad047f9ecf449bf72ce2bd # v3.0.0 | ||
# with: | ||
# # Paths to files will be available in `${FILTER_NAME}_files` output variable. | ||
# list-files: json | ||
# filters: | | ||
# all: | ||
# - added|modified: '**/*' | ||
# | ||
# # TODO: split out requirements-dev.txt | ||
# - uses: getsentry/action-setup-venv@a133e6fd5fa6abd3f590a1c106abda344f5df69f # v2.1.0 | ||
# with: | ||
# python-version: 3.11.8 | ||
# cache-dependency-path: | | ||
# requirements.txt | ||
# install-cmd: pip install -r requirements-precommit.txt | ||
# - name: Setup pre-commit | ||
# run: | | ||
# pre-commit install | ||
# | ||
# - name: Run pre-commit on PR commits | ||
# id: pre-commit_results | ||
# continue-on-error: true | ||
# run: | | ||
# jq '.[]' --raw-output <<< '${{steps.changes.outputs.all_files}}' | | ||
# # Run pre-commit to lint and format check files that were changed (but not deleted) compared to master. | ||
# xargs pre-commit run --files | ||
# | ||
# - name: Get auth token | ||
# id: token | ||
# if: ${{ steps.pre-commit_results.outcome == 'failure' }} | ||
# uses: getsentry/action-github-app-token@v3.0.0 | ||
# with: | ||
# app_id: ${{ vars.SENTRY_INTERNAL_APP_ID }} | ||
# private_key: ${{ secrets.SENTRY_INTERNAL_APP_PRIVATE_KEY }} | ||
# | ||
# - name: Apply any pre-commit fixed files | ||
# if: ${{ steps.pre-commit_results.outcome == 'failure' }} | ||
# # note: this runs "always" or else it's skipped when pre-commit fails | ||
# uses: getsentry/action-github-commit@v2.1.0 | ||
# with: | ||
# github-token: ${{ steps.token.outputs.token }} | ||
# message: ':hammer_and_wrench: apply pre-commit fixes' |