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

Fix linkchecker workflow and make file #4327

Merged
merged 7 commits into from
Apr 26, 2023

Conversation

germa89
Copy link
Contributor

@germa89 germa89 commented Apr 24, 2023

The workflow linkchecker is broken:

https://github.com/pyvista/pyvista/actions/workflows/linkcheck.yml

This PR should fix it. Some double check will be appreciated.

@github-actions github-actions bot added documentation Anything related to the documentation/website bug Uh-oh! Something isn't working as expected. labels Apr 24, 2023
@germa89 germa89 changed the title Fix make file Fix linkchecker workflow and make file Apr 24, 2023
@germa89 germa89 changed the title Fix linkchecker workflow and make file Fix linkchecker workflow and make file Apr 24, 2023
@germa89 germa89 changed the title Fix linkchecker workflow and make file Fix inkchecker workflow and make file Apr 24, 2023
@germa89 germa89 changed the title Fix inkchecker workflow and make file Fix linkchecker workflow and make file Apr 24, 2023
@akaszynski
Copy link
Member

Good catch.

Integration checks with MNE will be broken for a bit. Will merge once this has been fixed.

Agree with ' --> " for consistency with our extensions, but just FYI, we don't enforce single vs. double quotes.

@codecov
Copy link

codecov bot commented Apr 24, 2023

Codecov Report

Merging #4327 (f14057b) into main (b9ee9b9) will decrease coverage by 2.14%.
The diff coverage is n/a.

❗ Current head f14057b differs from pull request most recent head 7800be1. Consider uploading reports for the commit 7800be1 to get more accurate results

@@            Coverage Diff             @@
##             main    #4327      +/-   ##
==========================================
- Coverage   95.78%   93.65%   -2.14%     
==========================================
  Files          97       97              
  Lines       20773    20773              
==========================================
- Hits        19897    19454     -443     
- Misses        876     1319     +443     

doc/source/conf.py Outdated Show resolved Hide resolved
@akaszynski akaszynski merged commit 9038cee into pyvista:main Apr 26, 2023
20 checks passed
@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
bug Uh-oh! Something isn't working as expected. documentation Anything related to the documentation/website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants