-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Closing issue #2312 #2942
base: devel
Are you sure you want to change the base?
Closing issue #2312 #2942
Conversation
created docs of Clara inside, fixing issue catchorg#2312
updated Clara docs with link to new docs inside the catch documentation itself. Fixing issue catchorg#2312
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## devel #2942 +/- ##
=======================================
Coverage 90.99% 90.99%
=======================================
Files 198 198
Lines 8597 8597
=======================================
Hits 7822 7822
Misses 775 775 |
@@ -110,7 +110,7 @@ int main( int argc, char* argv[] ) { | |||
} | |||
``` | |||
|
|||
See the [Clara documentation](https://github.com/catchorg/Clara/blob/master/README.md) | |||
See the [Clara documentation](https://github.com/irvincardoza/Catch2/blob/devel/docs/Clara.md) |
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.
This link is reliant on your fork. Instead, it should be using relative reference to the new file.
[![Build Status](https://travis-ci.org/catchorg/Clara.svg?branch=master)](https://travis-ci.org/catchorg/Clara) | ||
[![Build status](https://ci.appveyor.com/api/projects/status/github/catchorg/Clara?brach=master&svg=true)](https://ci.appveyor.com/project/catchorg/clara) | ||
[![codecov](https://codecov.io/gh/catchorg/Clara/branch/master/graph/badge.svg)](https://codecov.io/gh/catchorg/Clara) |
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.
These are all irrelevant to the version embedded into the repo, as it is not built or tested separately.
Clara is a single-header library. | ||
|
||
To use, just `#include "clara.hpp"` |
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.
No longer true for the embedded version
Created a Clara Documentation inside the Catch2 docs. Also updated the link in the own-main docs to point to the new Clara docs