Skip to content

Conversation

@clatapie
Copy link
Contributor

Closes #1696

@clatapie clatapie self-assigned this Dec 20, 2022
@clatapie clatapie linked an issue Dec 20, 2022 that may be closed by this pull request
2 tasks
@github-actions
Copy link
Contributor

Please add one of the following labels to add this contribution to the Release Notes 👇

@codecov
Copy link

codecov bot commented Dec 21, 2022

Codecov Report

Merging #1722 (711d84f) into main (5e57a31) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1722      +/-   ##
==========================================
+ Coverage   85.26%   85.36%   +0.10%     
==========================================
  Files          45       45              
  Lines        7506     7498       -8     
==========================================
+ Hits         6400     6401       +1     
+ Misses       1106     1097       -9     

@clatapie clatapie marked this pull request as ready for review December 22, 2022 09:48
Copy link
Collaborator

@germa89 germa89 left a comment

Choose a reason for hiding this comment

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

Look good to me.

Just two comments:

  • Remember to fix Vale issues
  • Make sure you are not attaching extra files. Like examples/00-mapdl-examples/file.rst or doc/source/examples/gallery_examples/index.rst

clatapie and others added 9 commits December 22, 2022 11:15
Co-authored-by: German <28149841+germa89@users.noreply.github.com>
Co-authored-by: German <28149841+germa89@users.noreply.github.com>
Co-authored-by: German <28149841+germa89@users.noreply.github.com>
Co-authored-by: German <28149841+germa89@users.noreply.github.com>
Co-authored-by: German <28149841+germa89@users.noreply.github.com>
Co-authored-by: German <28149841+germa89@users.noreply.github.com>
@clatapie clatapie merged commit ac4dfb4 into main Dec 22, 2022
@clatapie clatapie deleted the maint/remove_vm_examples branch December 22, 2022 15:22
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.

Remove the Verification Manual Examples

3 participants