Skip to content

contribute/en/Release-Process: Document changelog helper jamulussoftware/jamulus#2257#708

Merged
ann0see merged 1 commit intojamulussoftware:releasefrom
hoffie:add-changelog-helper-pr2257
Mar 8, 2022
Merged

contribute/en/Release-Process: Document changelog helper jamulussoftware/jamulus#2257#708
ann0see merged 1 commit intojamulussoftware:releasefrom
hoffie:add-changelog-helper-pr2257

Conversation

@hoffie
Copy link
Copy Markdown
Member

@hoffie hoffie commented Feb 20, 2022

Changes

Replaces the instructions for manual changelog updates in the release check list by a reference to a new section which describes how to use the changelog helper from jamulussoftware/jamulus#2257.

Does this need translation?

  • Yes
  • No

PR is raised against release.

Related issues

jamulussoftware/jamulus#2257

@gilgongo
Copy link
Copy Markdown
Member

gilgongo commented Feb 24, 2022

Has this conflicted with my moving the website translation to after that of the app? If so, shall I put "Update the Changelog" after the website stuff has been done? So it would be:

  • Check for conflicting accelerator keys (see tools/checkkeys.pl)
    • Generate .qm files via lrelease Jamulus.pro
  • Finish Website translations
    • Wait for all PRs to be merged (missing translations will revert to English automatically)
    • Check for broken links with a link checker locally (run _po4a-tools/po4a-create-all-targets.sh locally first)
  • Check the milestone for mergable stuff again
  • Update the Changelog
  • Tag a release candidate... etc.

@hoffie hoffie force-pushed the add-changelog-helper-pr2257 branch from 84a2abc to 9b2cedc Compare February 24, 2022 23:08
@hoffie
Copy link
Copy Markdown
Member Author

hoffie commented Feb 24, 2022

Has this conflicted with my moving the website translation to after that of the app?

Yep, seems so.

I've just rebased. Marking this PR as Draft though as it depends on the actual feature being merged first.

@hoffie hoffie marked this pull request as draft February 24, 2022 23:09
@ann0see
Copy link
Copy Markdown
Member

ann0see commented Mar 8, 2022

Is this ready now?

@hoffie hoffie marked this pull request as ready for review March 8, 2022 20:27
@hoffie
Copy link
Copy Markdown
Member Author

hoffie commented Mar 8, 2022

Is this ready now?

I'd say so. Leaving merging to @ann0see or @gilgongo.

@ann0see ann0see merged commit 00f3c9b into jamulussoftware:release Mar 8, 2022
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