Test doxygen on alma10#22321
Draft
hageboeck wants to merge 8 commits into
Draft
Conversation
hageboeck
force-pushed
the
doxygen_alma10
branch
4 times, most recently
from
71dee01 to
3468ea1
Compare
hageboeck
added
skip ci
Test Results0 tests 0 ✅ 0s ⏱️ Results for commit a102b67. ♻️ This comment has been updated with latest results. |
hageboeck
force-pushed
the
doxygen_alma10
branch
2 times, most recently
from
a6bf834 to
02dfba3
Compare
- Base the doxygen job on the alma10 container. - This allows for using a preinstalled doxygen in the container (currently 1.13.2), replacing the manual install of 1.10 - Also use preinstalled XRootD and qhelpgenerator
- Remove the override step from the doxygen job. Overrides can be specified as command-line arguments, which significantly shortens the job file. - Remove duplicated build step. There is no difference between the jobs, so we don't need build job based on the workflow trigger. - Create & compress the tar archive in one go.
hageboeck
force-pushed
the
doxygen_alma10
branch
2 times, most recently
from
de82b04 to
0c2d436
Compare
- Add two job inputs for the workflow trigger. These should allow for choosing a ROOT version to build docs for, and for choosing a directory where the generated page should be served. Omitting the page means no upload to eos. - Remove the input "incremental". Since the documentation jobs don't upload or download anything, it never had an effect.
- Quote many variables to prevent word splitting or unwanted globbing
When the workflow file or the doxygen folder are touched, the CI will also run the doxygen build.
Strings such as "\s" are not correct, any more.
The md5 and map files are used only when doxygen is run on an existing folder. They therefore should not be uploaded or put in the tar archive.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.