Skip to content

Remove bump-version, make-release#339

Merged
crawfxrd merged 2 commits intomasterfrom
version-release
Oct 20, 2025
Merged

Remove bump-version, make-release#339
crawfxrd merged 2 commits intomasterfrom
version-release

Conversation

@crawfxrd
Copy link
Member

@crawfxrd crawfxrd commented Sep 24, 2025

Cut the cruft for making releases.

  • Use debchange for generating the debian/changelog entry instead of manually creating it through a Python script
  • Jenkins CI will tell us if building the package fails, so don't require doing it when making a release
  • Run tests as part of CI instead of when making a release

Note: Because GHA runs headless, GTK tests are not run.

@crawfxrd crawfxrd force-pushed the version-release branch 4 times, most recently from b0c76f0 to 3580e3f Compare September 26, 2025 19:27
@crawfxrd crawfxrd requested review from a team September 26, 2025 19:32
@crawfxrd crawfxrd marked this pull request as ready for review September 26, 2025 19:32
Copy link
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My only concern with this would be that we're not great about updating debian/changelog in most of our repos. As long as we can keep doing that consistently for this repo, this seems simpler than the two-script setup (and safer, as it doesn't require someone to push straight to master after every PR). The CI test appears to be working.

Copy link
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, please add making a tag after merging to the end of the new checklist. That's something else make-release.py did that's important for non-Pop!_OS/Ubuntu distros.

Cut the cruft for making releases.

- Use debchange for generating the `debian/changelog` entry instead of
  manually creating it through a Python script
- Jenkins CI will tell us if building the package fails, so don't
  require doing it when making a release
- Run tests as part of CI instead of when making a release

Signed-off-by: Tim Crawford <tcrawford@system76.com>
@crawfxrd
Copy link
Member Author

I've additionally documented it should be a signed tag.

jacobgkau
jacobgkau previously approved these changes Sep 29, 2025
@jacobgkau
Copy link
Member

@jackpot51 What do you think of this?

@crawfxrd crawfxrd merged commit a6e001d into master Oct 20, 2025
20 of 24 checks passed
@crawfxrd crawfxrd deleted the version-release branch October 20, 2025 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants