swaroopch / edn_format

EDN reader and writer implementation in Python, using PLY (lex, yacc)
https://swaroopch.com/2012/12/24/edn-format-python/
Other
131 stars 31 forks source link

Tweak Python versions in CI #90

Closed bfontaine closed 2 months ago

bfontaine commented 2 months ago

This should fix the build for https://github.com/swaroopch/edn_format/pull/89.

bfontaine commented 2 months ago

This seems to be stuck: it’s waiting for the build on 2.7, but I removed it so it will never run.

Edit: I opened this discussion about the issue.

bfontaine commented 2 months ago

I was thinking that we could keep the Python 2.7 support as a "best effort": remove it from the CI but keep the code for it, at least for now.

swaroopch commented 2 months ago

I was thinking that we could keep the Python 2.7 support as a "best effort": remove it from the CI but keep the code for it, at least for now.

I'm ok with that 👍🏽

bfontaine commented 2 months ago

@swaroopch I think this is stuck; the only way I see to fix this is to temporarily unprotect master, merge the PR, then protect it again.

swaroopch commented 2 months ago

Force merged, let me know if that helps.

bfontaine commented 2 months ago

Thanks!