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

✨ Add show_faces to geometry plotting tutorial #1643

Merged
merged 1 commit into from
Nov 17, 2022

Conversation

je-cook
Copy link
Contributor

@je-cook je-cook commented Nov 17, 2022

Description

I was asked to add a little for geometry plotting example only showing the wires not the faces

Interface Changes

None

Checklist

I confirm that I have completed the following checks:

  • Tests run locally and pass pytest tests --reactor
  • Code quality checks run locally and pass flake8 and black .
  • Documentation built locally and checked sphinx-build -W documentation/source documentation/build

@je-cook je-cook requested a review from a team as a code owner November 17, 2022 10:45
@je-cook je-cook closed this Nov 17, 2022
@je-cook je-cook reopened this Nov 17, 2022
@sonarcloud
Copy link

sonarcloud bot commented Nov 17, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@je-cook je-cook added examples Tasks relating to the examples / user training geometry Tasks relating to the geometry module labels Nov 17, 2022
@je-cook je-cook merged commit ebec2de into develop Nov 17, 2022
@github-actions
Copy link
Contributor

⚠️ Warning Report

Found 0 warnings. 🎉

@je-cook je-cook deleted the je-cook/geom-plot-tutorial branch November 17, 2022 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Tasks relating to the examples / user training geometry Tasks relating to the geometry module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants