Skip to content
This repository was archived by the owner on Oct 28, 2021. It is now read-only.

Conversation

@timothyqiu
Copy link

This PR

  • adds an interface library target Clara, aliased Clara::Clara

  • corresponding install entries are also added

    clara-install-prefix/
    |-- include
    |   `-- clara.hpp
    `-- lib
        `-- cmake
            `-- Clara
                |-- ClaraConfig.cmake
                `-- ClaraConfigVersion.cmake
    

    so find_package(Clara REQUIRED) can properly import Clara::Clara from the installed package.

  • include directories needed for ClaraTest targets are now set using target_include_directories instead of changing global settings

This implements the main part of #44

@timothyqiu
Copy link
Author

😞 One CI job failed because it can't pull code from GitHub, I don't know how to trigger that job again...

@codecov
Copy link

codecov bot commented May 2, 2019

Codecov Report

Merging #76 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #76   +/-   ##
=======================================
  Coverage   94.85%   94.85%           
=======================================
  Files           2        2           
  Lines         680      680           
=======================================
  Hits          645      645           
  Misses         35       35
Impacted Files Coverage Δ
include/clara.hpp 93.3% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6dfbbd...ba9db5b. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants