Skip to content
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 the GitHub Actions workflow #72

Merged

Conversation

mcdonnnj
Copy link
Contributor

@mcdonnnj mcdonnnj commented Feb 24, 2023

Description

This adds support for testing Python 3.11 and fixes the configuration for testing Python 3.6. It also does a bit of white space cleanup, applies the changes from the latest version of black, fixes new warnings from bandit, and updates the version of markdownlint-cli in the GitHub Actions workflow to the latest version.

@mcdonnnj mcdonnnj requested a review from IanLee1521 March 13, 2023 06:04
@mcdonnnj mcdonnnj force-pushed the improvement/update_gha_testing_configuration branch from 2e79b73 to 9d6bd10 Compare March 13, 2023 20:01
Specify the ubuntu-20.04 GitHub Actions runner image for the Python 3.6
test. GitHub does not make available Python 3.6 tarballs for
ubuntu-22.04 runners so we must use the older image for this test.
Reformat all Python files using black 23.1.0 as the project does not
currently pass a `make test` with a fresh installation.
Remove any trailing whitespace and ensure that all text files
appropriately end in newlines.
This updates the GitHub Actions workflow to use the latest version.
With the release of version 1.7.5 the bandit package now flags requests
calls (get(), post(), etc.) as medium severity security risks if they
do not have a timeout value provided. The default values we create are
the same as the defaults used by the github3.py package. This provides
as much parity as possible with other uses of the requests package in
this project.
@mcdonnnj mcdonnnj force-pushed the improvement/update_gha_testing_configuration branch from 9d6bd10 to 77b5047 Compare March 22, 2023 02:45
@IanLee1521 IanLee1521 merged commit 29ad21a into LLNL:main Mar 22, 2023
@mcdonnnj mcdonnnj deleted the improvement/update_gha_testing_configuration branch March 22, 2023 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants