Skip to content

feat(ci): Add doxygen to code quality workflow#356

Open
0utd4ted wants to merge 1 commit into
bao-project:mainfrom
0utd4ted:feat/ci/doxygen
Open

feat(ci): Add doxygen to code quality workflow#356
0utd4ted wants to merge 1 commit into
bao-project:mainfrom
0utd4ted:feat/ci/doxygen

Conversation

@0utd4ted

Copy link
Copy Markdown
Contributor

@0utd4ted 0utd4ted changed the title feat(ci): Add doxygen to code quality workflow. feat(ci): Add doxygen to code quality workflow Apr 16, 2026
@danielRep danielRep self-assigned this May 12, 2026
@0utd4ted

Copy link
Copy Markdown
Contributor Author

My signature has been nulled. I will force it up again, if it's not an issue. Since it's only 1 commit, can we just rebase at merge?

@danielRep

Copy link
Copy Markdown
Member

My signature has been nulled. I will force it up again, if it's not an issue. Since it's only 1 commit, can we just rebase at merge?

My bad. Yeah go ahead @AAGandomi

Signed-off-by: Aras Ashraf Gandomi <aras.gandomi@gmail.com>
@danielRep

Copy link
Copy Markdown
Member

FYI, bao-project/bao-ci#92 is merged. We are just missing #307.

@josecm josecm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The check we are introducing is failing itself. I think we may have to directly specify the doxyfile in the command.

Also, should we introduce this check as part of @bao-project/ci, and then call the ci make rule from this action?

@0utd4ted

0utd4ted commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

@josecm

The check we are introducing is failing itself. I think we may have to directly specify the doxyfile in the command.

Also, should we introduce this check as part of @bao-project/ci, and then call the ci make rule from this action?

This PR introduced Doxygen in the CI: bao-project/bao-ci#92 .
The Doxyfile, even if specified in the command, is missing (that is the reason why it's currently failing).
Doxygen will look automatically in the doxyfile in the root directory, so I don't think it's necessary to specify it in the CI command line.
A Doxyfile is being introduced as part of #307 (this PR has it as a dependency).

Edit: clarity.

@josecm

josecm commented Jun 25, 2026

Copy link
Copy Markdown
Member

@0utd4ted Sorry for the confusion! I haven't had time to look through these PRs for quite a while, so this wasn't fresh in my mind. I'll go through #307 over the next couple of weeks so we can get these moving.

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.

3 participants