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

Enable vale #4281

Merged
merged 1 commit into from
Apr 15, 2023
Merged

Enable vale #4281

merged 1 commit into from
Apr 15, 2023

Conversation

akaszynski
Copy link
Member

Enable Vale again.

@github-actions github-actions bot added documentation Anything related to the documentation/website maintenance Low-impact maintenance activity labels Apr 15, 2023
@codecov
Copy link

codecov bot commented Apr 15, 2023

Codecov Report

Merging #4281 (cdf21ab) into main (0c03106) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #4281   +/-   ##
=======================================
  Coverage   95.77%   95.77%           
=======================================
  Files          97       97           
  Lines       20755    20755           
=======================================
  Hits        19879    19879           
  Misses        876      876           

@tkoyama010
Copy link
Member

Since this CI change does not affect the pyvista code, I will merge without applying the 24 hour rule.

@tkoyama010 tkoyama010 merged commit 664a425 into main Apr 15, 2023
23 checks passed
@tkoyama010 tkoyama010 deleted the ci/enable-vale branch April 15, 2023 06:36
@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
documentation Anything related to the documentation/website maintenance Low-impact maintenance activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants