Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

Do not use amalgamated header for developing #149

Merged
merged 1 commit into from Dec 5, 2019
Merged

Conversation

bassosimone
Copy link
Member

@bassosimone bassosimone commented Dec 5, 2019

Rationale of the PR: when you develop you edit headers that make libndt.hpp. So, let's build tests and the real binary with headers that make libndt.hpp. Then, let's make sure we are also able to generate a libndt.hpp header that compiles, so we don't break it.

@codecov-io
Copy link

codecov-io commented Dec 5, 2019

Codecov Report

Merging #149 into master will decrease coverage by 3.29%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #149     +/-   ##
=========================================
- Coverage   88.23%   84.94%   -3.3%     
=========================================
  Files           5        6      +1     
  Lines        1513     2949   +1436     
=========================================
+ Hits         1335     2505   +1170     
- Misses        178      444    +266
Impacted Files Coverage Δ
impl.hpp 78.73% <ø> (ø)
libndt-client.cpp 43.75% <ø> (ø) ⬆️
libndt_test.cpp 94.05% <ø> (ø) ⬆️
curlx_test.cpp 100% <ø> (ø) ⬆️
sys.hpp 97.01% <0%> (+60.09%) ⬆️

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 ec2f1df...cd2283b. Read the comment docs.

@bassosimone bassosimone self-assigned this Dec 5, 2019
@bassosimone bassosimone added the enhancement New feature or request label Dec 5, 2019
@bassosimone
Copy link
Member Author

FTR @robertodauria I also fixed the too many builds by disabling this repo from travis-ci.org, which is the legacy CI from Travis, so we only build at travis-ci.com.

Copy link
Contributor

@robertodauria robertodauria left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@bassosimone bassosimone merged commit 06e6488 into master Dec 5, 2019
@bassosimone bassosimone deleted the develop branch December 5, 2019 18:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants