Skip to content

WIP: Trying building Doxygen on CircleCI for ITK#41

Closed
blowekamp wants to merge 2 commits intoInsightSoftwareConsortium:masterfrom
blowekamp:DoxygenOnCircleCI
Closed

WIP: Trying building Doxygen on CircleCI for ITK#41
blowekamp wants to merge 2 commits intoInsightSoftwareConsortium:masterfrom
blowekamp:DoxygenOnCircleCI

Conversation

@blowekamp
Copy link
Member

No description provided.

@blowekamp
Copy link
Member Author

@thewtex
Any idea why CircleCI is not build pull requests on Github? I wanted to see of ITK Doxygen could be build with in the time window allowed.

@thewtex
Copy link
Member

thewtex commented Mar 12, 2018

@blowekamp We have Build forked pull requests enabled in the InsightSoftwareConsortium/ITK CircleCI settings, which should enable this behavior. But, I have observed other projects not building their pull requests over the last few days, with no changes on my end ...

@blowekamp blowekamp force-pushed the DoxygenOnCircleCI branch 18 times, most recently from 65dc477 to 6057da1 Compare March 14, 2018 14:42
@blowekamp
Copy link
Member Author

@thewtex It looks like the build time for Doxygen on CircleCI should be doable. Should we persue building Doxygen on CircleCI further?

Currently, the CircleCI builds a failing due to compiler warnings. The docs job needs some work to doable testing. Yaml anchors need to be used to reduce dupolication etc...

@thewtex
Copy link
Member

thewtex commented Mar 14, 2018

@blowekamp If you would like to pursue it, then go for it.

@blowekamp
Copy link
Member Author

I can pursue getting this CircleCI script functioning, but it will need coordination on either getting itk.org updated with the results or figuring out how to push it to GitHub.io on demand.

@thewtex
Copy link
Member

thewtex commented Mar 15, 2018

I will look into using the results on itk.org if it works.

@blowekamp blowekamp force-pushed the DoxygenOnCircleCI branch 2 times, most recently from 9cb5541 to 3819166 Compare March 16, 2018 17:52
Avoid Doxygen warning about documented symbol not being declared.

Change-Id: I3e9d0a07cbe253dde2126b9f1c19b77ee3b9e2ad
@blowekamp blowekamp force-pushed the DoxygenOnCircleCI branch 4 times, most recently from c53f072 to 23103bb Compare March 27, 2018 17:26
Change-Id: Iac2fc9a3ce05dd1a9f11048a1410b7ad3de9e5a5
@hjmjohnson
Copy link
Member

@blowekamp can you rebase this onto of the master branch?

@kwrobot kwrobot added temp and removed temp labels Nov 6, 2018
@blowekamp
Copy link
Member Author

There is not longer interest in building Doxygen on CircleCI.

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.

4 participants