-
Notifications
You must be signed in to change notification settings - Fork 8
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
doc: remove notes containing syntax errors #658
Conversation
Instructions for code reviewerHello reviewer, thanks for taking the time to review this PR!
Checklist
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #658 +/- ##
=======================================
Coverage 78.78% 78.78%
=======================================
Files 36 36
Lines 1975 1975
Branches 141 141
=======================================
Hits 1556 1556
Misses 376 376
Partials 43 43 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Merge in when you see fit @Bai-Li-NOAA
What is the feature?
doc and style
GHA (see log here)to fail on the main branch.How have you implemented the solution?
Does the PR impact any other area of the project, maybe another repo?