Status of the project about caption HOT 3 OPEN

The ContentHack branch was working towards my vision of allowing a configurable caption for any content based on the original concept. I refactored the code so it made more sense in that context. Unfortunately, my python OOP skills are inadequate and I've stalled.

iirc:

1. I understand a superclass is needed once the content type is determined but I couldn't figure it out
2. two lines are written in the wrong order for tables

Of course if it worked then tests would be needed.

Any PR to resolve these or any other issues would be welcome.

from caption.

That's wonderful to hear (well, read 😅 )!

OK, now I understand what you tried to do, and while my PR fixes all the tests, they aren't really good from an OOP perspective. But at least we now know that it works and that a refactor should always pass the tests.

We can make the various content types subclasses of the CaptionTreeprocessor and register each one in the extendMarkdown method, so that the parent class only has common code and the children only have specialized code.

I would gladly help you do this, let me know if you want to merge the current PR first or if it's ok to go all in with the overhaul.

PS: I already did some more refactorings, mainly to make it more compliant with the google style guide (variable/method/class naming) and to switch to pytest (just a personal preference, it avoids keeping a useless class around), but let's take it one step at a time 😉

from caption.

Small note in the wiki on page https://github.com/flywire/caption/wiki/Development#unittest

the line
python -m unittest -v test_caption.captionTestCase

should be replaced by
pytest --cov

As the testing framework was changed.

from caption.