Skip to content

Commit 6675a7d

Browse files
authored
Move problem matchers step back
@kattni, I'm changing this back via patch because I ended up checking out what this step does and the instructions behind it say it should go before the tests (I assume `pre-commit` tests). I didn't realize it's intended to help visually show where the errors are. I'll make sure this is contained in the library patch because I think some libraries have it as the last step, which I think means it won't work.
1 parent ef5e3f7 commit 6675a7d

File tree

1 file changed

+2
-2
lines changed
  • {{ cookiecutter and 'tmp_repo' }}/.github/workflows

1 file changed

+2
-2
lines changed

{{ cookiecutter and 'tmp_repo' }}/.github/workflows/build.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,8 @@ jobs:
4747
pip install --force-reinstall Sphinx sphinx-rtd-theme pre-commit
4848
- name: Library version
4949
run: git describe --dirty --always --tags
50+
- name: Setup problem matchers
51+
uses: adafruit/circuitpython-action-library-ci-problem-matchers@v1
5052
- name: Pre-commit hooks
5153
run: |
5254
pre-commit run --all-files
@@ -76,5 +78,3 @@ jobs:
7678
python setup.py sdist
7779
python setup.py bdist_wheel --universal
7880
twine check dist/*
79-
- name: Setup problem matchers
80-
uses: adafruit/circuitpython-action-library-ci-problem-matchers@v1

0 commit comments

Comments
 (0)