-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update options documentation #5083
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #5083 +/- ##
=======================================
Coverage 76.2% 76.2%
=======================================
Files 760 760
Lines 61489 61489
Branches 8115 8113 -2
=======================================
+ Hits 46832 46839 +7
+ Misses 14657 14650 -7 |
Co-authored-by: Elliot Lee <github.public@intelliot.com>
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.
👍
BUILD.md
Outdated
| `coverage` | OFF | Prepare the coverage report. | | ||
| `tests` | ON | Build tests. | | ||
| `unity` | ON | Configure a unity build. | | ||
| `reporting` | OFF | Build the reporting mode feature. | |
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.
I assume this is later removed in the PR that removes reporting mode?
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.
Yes, though it will need to be merged with this change after it hits develop
.
No description provided.