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 notes in doc about mesh parser #487

Merged
merged 6 commits into from
Sep 26, 2022
Merged

Add notes in doc about mesh parser #487

merged 6 commits into from
Sep 26, 2022

Conversation

KnutAM
Copy link
Member

@KnutAM KnutAM commented Sep 26, 2022

After discussions on FerriteCon, good to highlight that there are packages that can do this.

Currently adding in manual + in the first example.

@KnutAM KnutAM requested a review from kimauth September 26, 2022 14:29
@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2022

Codecov Report

Base: 91.91% // Head: 91.91% // No change to project coverage 👍

Coverage data is based on head (b80de13) compared to base (64efcf1).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #487   +/-   ##
=======================================
  Coverage   91.91%   91.91%           
=======================================
  Files          22       22           
  Lines        3647     3647           
=======================================
  Hits         3352     3352           
  Misses        295      295           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@KnutAM KnutAM merged commit 6cbf339 into master Sep 26, 2022
@KnutAM KnutAM deleted the kam/meshdoc branch September 26, 2022 17:12
@fredrikekre
Copy link
Member

Preferably squash-merge when there are a bunch of "fixup" commits like here :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants