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

Test with Python 3.11 #3944

Merged
merged 1 commit into from
Feb 6, 2023
Merged

Test with Python 3.11 #3944

merged 1 commit into from
Feb 6, 2023

Conversation

banesullivan
Copy link
Member

@mwtoews, this handles adding Python 3.11 to CI to supersede some of your changes in #3938

Adds CI testing for Python 3.11 and attempts to speed up the docs build with 3.11

@github-actions github-actions bot added the maintenance Low-impact maintenance activity label Feb 2, 2023
Comment on lines -177 to -186
- name: Publish notebooks on main
if: github.event_name == 'push' && github.ref == 'refs/heads/main'
uses: peaceiris/actions-gh-pages@v3
with:
external_repository: pyvista/pyvista-examples
personal_token: ${{ secrets.PYVISTA_BOT_TOKEN }}
publish_dir: pyvista-examples
publish_branch: develop
exclude_assets: ''
force_orphan: true
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are pushing to the same repo as the release and thus overwriting the examples from the release. The docker image there uses the latest tag so it's very likely that the examples will mismatch the PyVista version there

@codecov
Copy link

codecov bot commented Feb 2, 2023

Codecov Report

Merging #3944 (66abdd0) into main (c23df22) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3944   +/-   ##
=======================================
  Coverage   94.30%   94.30%           
=======================================
  Files          94       94           
  Lines       20321    20321           
=======================================
  Hits        19163    19163           
  Misses       1158     1158           

@banesullivan
Copy link
Member Author

There are pending status checks because some of these are set as required in the repo settings. If this merges, I will update the required checks to the new jobs added here

@banesullivan banesullivan merged commit 2484730 into main Feb 6, 2023
@banesullivan banesullivan deleted the ci/py311 branch February 6, 2023 20:31
@banesullivan banesullivan mentioned this pull request Feb 6, 2023
@akaszynski akaszynski mentioned this pull request Apr 30, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Low-impact maintenance activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants