-
Notifications
You must be signed in to change notification settings - Fork 100
Fixes and Cleanup of CI #469
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
+307
−793
Merged
Changes from all commits
Commits
Show all changes
12 commits
Select commit
Hold shift + click to select a range
6d7880b
Fix missing pdm installs in release workflow
sjmonson 2c74b6c
Drop broken link-check
sjmonson 5e18bb0
Consolidate test jobs
sjmonson 07a8b58
Mark a few e2e tests as sanity
sjmonson cff1deb
Consolidate main test jobs
sjmonson 0fb948a
Consolidate quality jobs
sjmonson 0b8f89c
testing-ui -> ui-testing
sjmonson e28512c
Move tox run to custom action
sjmonson dc845e8
Use PDM python installer action
sjmonson 52a11ed
Run full test suite on PRs
sjmonson ff4e64d
Reduce tests run in main to smoke and only run PR checks on oldest ve…
sjmonson ee1977f
Ignore errno 5 for tox/pytest
sjmonson File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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 |
|---|---|---|
| @@ -0,0 +1,29 @@ | ||
| name: 'Run tox with environment' | ||
| description: 'Runs tox with pdm as the package manager' | ||
| inputs: | ||
| python-version: | ||
| type: string | ||
| required: true | ||
| tox-env: | ||
| type: string | ||
| required: true | ||
| tox-args: | ||
| type: string | ||
| required: false | ||
| runs: | ||
| using: "composite" | ||
| steps: | ||
| - name: Setup Python with PDM | ||
| uses: pdm-project/setup-pdm@v4 | ||
| with: | ||
| python-version: ${{ inputs.python-version }} | ||
| - name: Install dependencies | ||
| run: | | ||
| pip install tox tox-pdm | ||
| shell: bash | ||
| - name: Run tox | ||
| run: | | ||
| tox run -e "${{ inputs.tox-env }}" -- ${{ inputs.tox-args }} | ||
| # errno 5 means all tests were filtered out, ignore | ||
| ret=$? && [[ $ret -eq 5 ]] && exit 0; exit $ret | ||
| shell: bash | ||
This file contains hidden or 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
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.