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

Closing issue #2312 #2942

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from
Open

Closing issue #2312 #2942

wants to merge 2 commits into from

Conversation

irvincardoza
Copy link

Created a Clara Documentation inside the Catch2 docs. Also updated the link in the own-main docs to point to the new Clara docs

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
Copy link

codecov bot commented Dec 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.99%. Comparing base (0321d2f) to head (866d4f9).

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)
Copy link
Member

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.

Comment on lines +2 to +4
[![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)
Copy link
Member

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.

Comment on lines +11 to +13
Clara is a single-header library.

To use, just `#include "clara.hpp"`
Copy link
Member

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

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

Successfully merging this pull request may close these issues.

2 participants