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.

Sign up for GitHub

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

Jump to bottom

fix the doxygen ci build job Nr. 2 #1039

Merged
merged 1 commit into from
Jun 15, 2020
Merged

fix the doxygen ci build job Nr. 2 #1039

merged 1 commit into from
Jun 15, 2020

Conversation

SimeonEhrig
Copy link
Member

The PR #1038 doesn't fixed the problem.

@SimeonEhrig SimeonEhrig changed the title fix the doxygen ci build job fix the doxygen ci build job #2 Jun 11, 2020
@SimeonEhrig SimeonEhrig changed the title fix the doxygen ci build job #2 fix the doxygen ci build job Nr. 2 Jun 11, 2020
@SimeonEhrig SimeonEhrig changed the title fix the doxygen ci build job Nr. 2 [WIP] fix the doxygen ci build job Nr. 2 Jun 11, 2020
@SimeonEhrig
Copy link
Member Author

@BenjaminW3
I could run the doxygen job in my private repo: https://github.com/SimeonEhrig/alpaka/runs/760845738
I tried to fix the problem, that git add /docs/doxygen/html was forbidden by the .gitignore. The test ran for a while, but crashed at adding files without an error message. Maybe it didn't catch the error message for git push, which can potentially fail because of missing permission. Do you have an idea?

BenjaminW3
BenjaminW3 reviewed Jun 11, 2020
View reviewed changes
.gitignore Outdated Show resolved Hide resolved
@BenjaminW3
Copy link
Member

It should be possible to git add --force the files even though they are hidden by .gitignore.

@SimeonEhrig
Copy link
Member Author

I tested the new solution in my private repository and the GitHub workflow crashed with the same (nondescripting) error message, like the old approach.

@SimeonEhrig
Copy link
Member Author

@BenjaminW3
If you fine with the solution, can you please merge.

@BenjaminW3 BenjaminW3 merged commit 043df1a into alpaka-group:develop Jun 15, 2020
@psychocoderHPC psychocoderHPC changed the title [WIP] fix the doxygen ci build job Nr. 2 fix the doxygen ci build job Nr. 2 Jun 22, 2020
This was referenced Jun 26, 2020
Closed
Merged
@psychocoderHPC psychocoderHPC added this to the Version 0.6.0 milestone Jan 4, 2021
@SimeonEhrig SimeonEhrig deleted the fixDoxygen2 branch September 28, 2021 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BenjaminW3 BenjaminW3 BenjaminW3 left review comments

Assignees
No one assigned
Labels
Type:Bug Type:Documentation
Projects
None yet
Milestone
Version 0.6.0
Development

Successfully merging this pull request may close these issues.
3 participants
@SimeonEhrig @BenjaminW3 @psychocoderHPC