openzim / gutenberg

Scraper for downloading the entire ebooks repository of project Gutenberg
https://download.kiwix.org/zim/gutenberg
GNU General Public License v3.0
126 stars 37 forks source link

Release container build should be triggered on release event #176

Closed kelson42 closed 1 year ago

kelson42 commented 1 year ago

… but for now this is on tagging event.

rgaudin commented 1 year ago

Fixed

kelson42 commented 1 year ago

This is not fixed by https://github.com/openzim/gutenberg/commit/d5235587c9d867a0fff905540670c8605e481218 which does do anything regarding container image building

rgaudin commented 1 year ago
kelson42 commented 1 year ago
  • We don't handle container builds on release anywhere.

Yes, we start now

  • Github release process is based on tags so it's equivalent

It's not, you can perfectly have a tag without a release.

The problem is that now we have a release process which is not atomic, one part runs first (on tagging) and one other part runs later (on release). This is uselessly complex and actually pretty error prone.

rgaudin commented 1 year ago

Yes, we start now

It would be so much better to introduce such changes in an overview ticket and pass it down instead of popping up in a random scraper without any justification.

kelson42 commented 1 year ago

@rgaudin We had a discussion about this ticket. Does that still need anything else?