8000 github: Fix issue with draft releases by jku · Pull Request #2389 · theupdateframework/python-tuf · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@jku
Copy link
Member
@jku jku commented May 10, 2023

Commit 707dc49 included a change where the release candidate was marked as draft. This was a mistake as draft releases are only visible to logged in maintainers. This leads to e.g. ./verify_release script failing while the release is a draft.

Revert those changes:

  • don't use "draft" attribute
  • postfix the release name with "-rc" while the release waits for approval
  • Only set the real description and name after release approval

Fixes #2388

@coveralls
Copy link
coveralls commented May 10, 2023

Pull Request Test Coverage Report for Build 4936605681

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.692%

Totals Coverage Status
Change from base Build 4936012491: 0.0%
Covered Lines: 1320
Relevant Lines: 1343

💛 - Coveralls

Copy link
Member
@lukpueh lukpueh left a comment

Choose a reason for hiding this comment

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

This should work.

Commit 707dc49 included a change where the release candidate was marked
as draft. This was a mistake as draft releases are only visible to
logged in maintainers. This leads to e.g. ./verify_release script
failing while the release is a draft.

Revert those changes:
* don't use "draft" attribute
* postfix the release name with "-rc" while the release waits for
  approval
* Only set the real description and name after release approval

Signed-off-by: Jussi Kukkonen <jkukkonen@google.com>
@jku jku force-pushed the fix-draft-release branch from 991461e to ffc9049 Compare May 10, 2023 11:47
@jku jku merged commit 8c7dfd1 into theupdateframework:develop May 10, 2023
@jku jku deleted the fix-draft-release branch December 30, 2024 09:14
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.

verify_release: Fix issue with draft releases no showing up

3 participants

0