marzer / poxy

Documentation generator for C++
https://pypi.org/project/poxy
MIT License
135 stars 5 forks source link

[WIP] Revert to using pytomlpp #25

Closed bobfang1992 closed 1 year ago

bobfang1992 commented 1 year ago

Want to try this to see what's going on

bobfang1992 commented 1 year ago

Ah sorry this seems not triggering the CI

marzer commented 1 year ago

this seems not triggering the CI

D:

It's supposed to! Lemme look into it. Guess I don't have this repo configured correctly.

EDIT: ok, I see, it's because requirements.txt isn't in the set of CI trigger paths. I'll push a fix for this later today.

marzer commented 1 year ago

@bobfang1992 it should be OK now, try force-pushing again

bobfang1992 commented 1 year ago

Hi do you mind approving the run? Thanks!

marzer commented 1 year ago

Interesting, it seems to work OK now O_o

I think I might leave it as using tomli for the time being since poxy isn't perf-critical in any meaningful way (it will still preferentially use pytomlpp if it's available)