search

svenevs / exhale

Automatic C++ library api documentation generation: breathe doxygen in and exhale it out.
BSD 3-Clause "New" or "Revised" License
222 stars 51 forks source link

Fix current tests #220

Open renefritze opened 3 months ago

renefritze commented 3 months ago

The current tests now all pass in my fork. Couple of notes:

  1. I've moved to using the codecov action because the deprecated upload immeadiately ran into rate limiting. This will need a CODECOV_TOKEN repo secret
  2. If you're not happy with the new lowest supported version, it's probably not too hard to fix the tests for it. The errors just looked like SPhinx internals that I knew nothing about.
  3. I've added a mostly unintrusive pre-commit setup, mostly to get actionlint going. Would you be happy with a more substantial config that also does linting and formatting (with ruff)?
  4. Locally the Fortran CPPFOrtranMixed.tests_hierarchies fails. For some reason the checked functions have an additional intent(in) argument. I've just ignored that.
renefritze commented 3 months ago

No worries, I'm not in a hurry. I'd be happy to spend a little more time on the actual concepts stuff, if you find that would be useful.

renefritze commented 2 months ago

Gentle poke @svenevs

svenevs commented 2 months ago

Gah I'm so sorry :/ I'm moving the next few days but have set a reminder for this weekend...

svenevs commented 2 months ago

Hi there, just wanted to check in, I had to move the reminders back, I need to finish getting settled before work tomorrow and unfortunately have a lot to catch up on this week.

But now that the dust is settling and I have a better living arrangement I will be able to get the needed facelift here through. I'm sorry this keeps falling off the radar, I've got numerous calendar reminders and sticky notes up.

Thank you for being patient, it's been a rough couple of months :/

renefritze commented 1 month ago

Hey, it's me again :wave: offering support in moving this along if needed