-
Notifications
You must be signed in to change notification settings - Fork 198
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
Doxygen CI: Warnings as Error #3560
base: development
Are you sure you want to change the base?
Conversation
bc910c2
to
d615120
Compare
d615120
to
69a2954
Compare
69a2954
to
d9bd694
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
72198c8
to
5568331
Compare
Found this old PR and just merged |
4492996
to
440d9ed
Compare
FYI, still struggling to "fix" this: /home/runner/work/WarpX/WarpX/Source/Utils/ParticleUtils.H:36: error: no matching file member found for
amrex::DenseBins< ParticleTileDataType > ParticleUtils::findParticlesInEachCell(int lev, const amrex::MFIter &mfi, WarpXParticleContainer::ParticleTileType &ptile)
Possible candidates:
'amrex::DenseBins< ParticleTileDataType > findParticlesInEachCell(int lev, const amrex::MFIter &mfi, ParticleTileType &ptile)' at line 40 of file /home/runner/work/WarpX/WarpX/Source/Utils/ParticleUtils.cpp
(warning treated as error, aborting now) |
Make sure that no Doxygen warnings get in, as we do in other CI runs already.
440d9ed
to
4e593df
Compare
I'd be happy to finalize this, unfortunately when I tried in December I ran into the weird "error" (which seems to me like a false positive) described in #3560 (comment) above, and I could not figure out how to fix it. We can see how it goes now, after the latest rebase. |
We could try to check if the |
Make sure that no Doxygen warnings get in, as we do in other CI runs already.
Depends on AMReX-Codes/amrex#3061