Infer token #82
Infer token #82
Conversation
This comment has been minimized.
This comment has been minimized.
| @@ -134,7 +133,6 @@ and removal, and `skipped tests` to detect new skipped and un-skipped tests, as | |||
| See this complete list of configuration options for reference: | |||
| ```yaml | |||
| with: | |||
| github_token: ${{ secrets.GITHUB_TOKEN }} | |||
There was a problem hiding this comment.
This lists all available options, and in some cases you might want to provide a different token than the default one. So this should stay for completeness.
There was a problem hiding this comment.
ah, good eye! i updated the name to your-custom-pat to try and emphasize that it isn't necessary to provide. happy to switch back to ${{ secrets.GITHUB_TOKEN }} or whatever other string you want.
| @@ -38,11 +38,10 @@ jobs: | |||
| python -m pytest --continue-on-collection-errors --junit-xml ../test-results/pytest.xml | |||
|
|
|||
| - name: Unit Test Results | |||
| uses: EnricoMi/publish-unit-test-result-action@master | |||
| uses: ./ | |||
There was a problem hiding this comment.
Getting this to work across branches and forks is great, this might even fix #45.
There was a problem hiding this comment.
yup, likely will 😃
This comment has been minimized.
This comment has been minimized.
|
@justAnotherDev can you delete the |
oops, done! 67b0d32 |
This comment has been minimized.
This comment has been minimized.
Unit Test Results 1 files ±0 1 suites ±0 1s ⏱️ -1s Results for commit 2d28b77. ± Comparison against base commit 56099e8. This pull request removes 1 and adds 3 tests. Note that renamed tests count towards both. |
Use the GitHub provided
${{ github.token }}as the default so the user does not need to provide one.Example of
actions/checkoutusing the same syntax.This update does NOT require existing users to change anything.
Note:
This PR also updates the CI file to use whatever the current fork/branch's code is instead of always using master. This allows other branches and forks to test before creating a PR.