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

Set up Gitlab to deploy doxygen code reference to website #105

Merged
merged 1 commit into from
Feb 2, 2021

Conversation

vvolkl
Copy link
Collaborator

@vvolkl vvolkl commented Jan 26, 2021

BEGINRELEASENOTES

  • Use gitlab CI to deploy doxygen site to eos

ENDRELEASENOTES

This repo is now mirrored to https://gitlab.cern.ch/key4hep/edm4hep (private repo), where it should be able to take advantage of the gitlab infrastructure to deploy doxygen to eos.

@vvolkl vvolkl changed the title [wip] web depoy site to eos using gitlab Set up Gitlab to deploy doxygen code reference to website Feb 2, 2021
@vvolkl
Copy link
Collaborator Author

vvolkl commented Feb 2, 2021

Not sure if the comments ended up a bit to verbose, but at least they should give fairly complete instructions.

@vvolkl vvolkl requested a review from andresailer February 2, 2021 11:01
@vvolkl
Copy link
Collaborator Author

vvolkl commented Feb 2, 2021

I already pushed this to the gitlab mirror to test it out, so https://cern.ch/edm4hep is already created from this.

@andresailer
Copy link
Collaborator

That picture with the EDM diagram doesn't show up
https://key4hep.github.io/EDM4hep/doc/edm4hep_diagram.svg

@vvolkl
Copy link
Collaborator Author

vvolkl commented Feb 2, 2021

That picture with the EDM diagram doesn't show up
https://key4hep.github.io/EDM4hep/doc/edm4hep_diagram.svg

Right, that was taken from the branch I just now deleted to fix the repository size. Should be fixed by linking directly to the file in the repo master branch.

@andresailer
Copy link
Collaborator

Did you check that the picture also shows up in the doxygen output now?

@vvolkl vvolkl force-pushed the webdeploy branch 2 times, most recently from 31293d7 to 37b8b1d Compare February 2, 2021 14:35
@vvolkl
Copy link
Collaborator Author

vvolkl commented Feb 2, 2021

@andresailer I had to add some lines to the cmake to make sure it the diagram is available in the build/install folder, but now it seems ok, see the deployment of this branch on cern.ch/edm4hep

@vvolkl
Copy link
Collaborator Author

vvolkl commented Feb 2, 2021

Also added a notice to https://key4hep.github.io/EDM4hep/

* Replaces the existing travis / github pages workflow for doxygen
* fixes the diagram and its installation
* renames the cmake option to controll the doc generation to BUILD_DOCS
  as this is more generic and widely used
Copy link
Collaborator

@andresailer andresailer left a comment

Choose a reason for hiding this comment

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

Thanks!

@vvolkl vvolkl merged commit 1bd0b5d into key4hep:master Feb 2, 2021
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