twolfson / restructuredtext-lint

reStructuredText linter
The Unlicense
171 stars 20 forks source link

Upgrade #59

Closed eyllanesc closed 2 years ago

eyllanesc commented 2 years ago
twolfson commented 2 years ago

Thanks for the PR =) Unfortunately, I don't want a lot of these changes =/

eyllanesc commented 2 years ago

I understand your points. I offer you the following alternative:

Do not remove travis but add GA. GA could be used for recent versions of python and others OS which doesn't seem to be covered in travis. The dependabot thing could be optional, I don't see a great advantage but not a big problem either, I will only create PR of updates. What do you think?

twolfson commented 2 years ago

Eh, we're not doing anything sophisticated/new language feature dependent, so testing early Python 3.x prob covers us quite well still. I think Travis CI is good enough for now

eyllanesc commented 2 years ago

My initial proposal was caused by seeing travis.ci badge in the README pointing out that there was a problem. Then I noticed that you were testing with versions of python that are no longer supported, so I thought that perhaps with new versions of python something could have been broken but it is not, but perhaps in the future it could happen. Anyway the changes depend on you. Goodbye, thanks for your time.

twolfson commented 2 years ago

Ah, I see. We just shifted from Travis CI org to com so it's likely related to that issue. Thanks for the heads up =)