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: Pyvista not included in doc-no-examples dependencies #4977

Merged
merged 9 commits into from
Jul 31, 2024

Conversation

Samuelopez-ansys
Copy link
Member

@Samuelopez-ansys Samuelopez-ansys commented Jul 31, 2024

When merging #4893 the step doc-build was not performed because it requires doc-style which couldn't run because too many changes were performed.

This led to a main branch with documentation extended and missing pyista dependency.

@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@github-actions github-actions bot added maintenance Package and maintenance related documentation Improvements or additions to documentation labels Jul 31, 2024
maxcapodi78
maxcapodi78 previously approved these changes Jul 31, 2024
@github-actions github-actions bot added the examples Anything related to the examples label Jul 31, 2024
@SMoraisAnsys
Copy link
Collaborator

Why would we need the pyvista dependency if we don't run the examples ?

Copy link

codecov bot commented Jul 31, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 83.53%. Comparing base (fb44e5a) to head (6e57d8a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4977      +/-   ##
==========================================
- Coverage   83.65%   83.53%   -0.12%     
==========================================
  Files         121      121              
  Lines       54791    54793       +2     
==========================================
- Hits        45838    45774      -64     
- Misses       8953     9019      +66     

@Samuelopez-ansys Samuelopez-ansys enabled auto-merge (squash) July 31, 2024 14:38
@Samuelopez-ansys Samuelopez-ansys merged commit 953f428 into main Jul 31, 2024
45 of 46 checks passed
@Samuelopez-ansys Samuelopez-ansys deleted the doc/sbrplus_doc branch July 31, 2024 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation examples Anything related to the examples maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants