Skip to content
This repository has been archived by the owner on Apr 13, 2021. It is now read-only.

Make Travis check doxygen #307

Merged
merged 2 commits into from
Mar 16, 2016
Merged

Make Travis check doxygen #307

merged 2 commits into from
Mar 16, 2016

Conversation

ljbade
Copy link
Contributor

@ljbade ljbade commented Feb 20, 2016

No description provided.

@ljbade ljbade force-pushed the travis-doxygen branch 4 times, most recently from b419b66 to 3759d11 Compare February 20, 2016 23:38
@ljbade
Copy link
Contributor Author

ljbade commented Feb 21, 2016

This PR will prevent future doxygen errors being overlooked before a merge.

Currently fails due to doxygen bug triggered by https://github.com/swift-nav/libswiftnav/pull/303

@ljbade
Copy link
Contributor Author

ljbade commented Feb 21, 2016

@jacobmcnamee I added a work around by adding a typedef for enum code to code_t and using that in the function definitions.

Are you OK with that?

@ljbade
Copy link
Contributor Author

ljbade commented Feb 21, 2016

Hmm there is something going on with coveralls, seems the hits/line went down?

@ljbade
Copy link
Contributor Author

ljbade commented Feb 28, 2016

@fnoble Can you look at why Coveralls is flagging this PR?

I believe @dt-exafore had same problems in https://github.com/swift-nav/libswiftnav/pull/308

@ljbade
Copy link
Contributor Author

ljbade commented Mar 4, 2016

Going to merge this PR and ignore the coveralls blip if no one complains.

ljbade added a commit that referenced this pull request Mar 16, 2016
@ljbade ljbade merged commit ec96142 into swift-nav:master Mar 16, 2016
@ljbade ljbade deleted the travis-doxygen branch March 16, 2016 01:10
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.

1 participant