JEFuller / Piexif

Exif manipulation with pure python script.
MIT License
12 stars 3 forks source link

CI migration #5

Closed davetapley closed 1 year ago

davetapley commented 2 years ago

Appveyor

Inherited

https://github.com/JEFuller/Piexif/blob/3422fbe7a12c3ebcc90532d8e1f4e3be32ece80c/appveyor.yml

https://ci.appveyor.com/project/hMatoba/piexif/history

Last pass Nov 2020: https://ci.appveyor.com/project/hMatoba/piexif/builds/36156177

Migration

Moved to https://ci.appveyor.com/project/davetapley/piexif

Travis

Inherited

https://github.com/hMatoba/Piexif/blob/3422fbe7a12c3ebcc90532d8e1f4e3be32ece80c/.travis.yml

No history: https://app.travis-ci.com/github/hMatoba/Piexif/builds

Migration

Moved to https://app.travis-ci.com/github/JEFuller/Piexif

But:

We are unable to start your build at this time. You exceeded the number of users allowed for your plan. Please review your plan details and follow the steps to resolution.

~awaiting response from Support support@travis-ci.com~ awaiting OSS build credits from Support support@travis-ci.com

Able to trigger build but lots of failures :x:

Coveralls

https://coveralls.io/github/hMatoba/Piexif

Last build Nov 2020: https://coveralls.io/builds/34747634

Triggered from: https://github.com/JEFuller/Piexif/blob/3422fbe7a12c3ebcc90532d8e1f4e3be32ece80c/.travis.yml#L22-L23

Not not sure how when no Travis history šŸ¤”

hartwork commented 1 year ago

Hi!

In reply to https://github.com/JEFuller/Piexif/issues/1#issuecomment-1374672884, I would like to summarize what I see in the existing CI files linked above:

I suggest to:

I'd like to add a personal vote for dropping Python 2.7 and 3.5 because of end-of-life. What do you think, what are the requirements?

PS: Here's the low-level bits for a handmade coveralls integration: https://github.com/asyncee/django-easy-select2/blob/f261c5b75145ed5203c500fd5b2fef177ec91d47/tox.ini#L34-L45 (with tox around, but that can be ignored)

davetapley commented 1 year ago

@hartwork sorry I missed this šŸ˜Š

hartwork commented 1 year ago
  • I concur that GitHub Actions rewrite is best. I'll abandon appveyor and Travis and work on that

@davetapley cool!

  • I'll see if 2.7 and 3.5 still work, but yes I think it's appropriate to drop them if they cause trouble

Not sure about trouble but surely cost, both CI and human: Everything before 3.7 is dead: https://endoflife.date/python

djjudas21 commented 1 year ago

Hey @davetapley, how are you getting on with this?

davetapley commented 1 year ago

@djjudas21 haven't even started, but I can make time to merge PRs!

djjudas21 commented 1 year ago

I've had a crack at porting to GitHub Actions in #7

davetapley commented 1 year ago

7