-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update coveralls-python Configuration in GitHub Actions Workflow #71
Update coveralls-python Configuration in GitHub Actions Workflow #71
Conversation
coveralls-python has better integration with running in GitHub Actions now, so we can use the automatically generated GITHUB_TOKEN secret instead of manually setting a COVERALLS_REPO_TOKEN secret for each repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved after a minor change. ⚙️
Please also see my one question.
.github/workflows/build.yml
Outdated
COVERALLS_SERVICE_NAME: github | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
COVERALLS_FLAG_NAME: "py${{ matrix.python-version }}" | ||
COVERALLS_PARALLEL: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sort environment variables alphabetically:
COVERALLS_SERVICE_NAME: github | |
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | |
COVERALLS_FLAG_NAME: "py${{ matrix.python-version }}" | |
COVERALLS_PARALLEL: true | |
COVERALLS_FLAG_NAME: "py${{ matrix.python-version }}" | |
COVERALLS_PARALLEL: true | |
COVERALLS_SERVICE_NAME: github | |
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
7f57cc0
to
30a1c14
Compare
We already ran our tests in parallel for each major version of Python a project supports. This change should better differentiate each test job run in the reports submitted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏆
🗣 Description
This PR updates the configuration for
coveralls-python
in our GitHub Actions workflow to use the automatically createdGITHUB_TOKEN
secret instead of a per-repoCOVERALLS_REPO_TOKEN
we have used previously to close #70. It also tags each individualtest
job (one per Python version supported) at https://coveralls.io by changing our configuration to reflect that the job runs in parallel.💭 Motivation and context
@dav3r ran into an issue with how
coveralls-python
is run in our GitHub Actions workflow in cisagov/gophish-tools#29. He determined the fix was the update the configuration to what was described in the documentation. I believe we have run afoul of changes incoveralls-python
v3 to the order in which configurations are processed.🧪 Testing
Automated tests run without issue, and I confirmed both that coverage reports were uploaded and that the tagging for parallel tasks is working.
✅ Checklist