Skip to content

Conversation

@iwilltry42
Copy link
Contributor

This PR adds release discussions to the plugin.

Commit Message:

  • bump GitHub dep to v35.3.0 as this was introduced in v35.1.0
  • add new string flag "discussion-category" (+ env vars)
  • update release: only set category, if not set
  • NOTE: Currently it looks like release drafts with discussion category
    set do not "persist" the category when being released, so when
    publishing a release draft, without explicitly setting the category, it
    won't have a release discussion. Nothing we can do here I suppose.

create_release
release_with_discussion

Copy link
Contributor

@donny-dont donny-dont left a comment

Choose a reason for hiding this comment

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

Rebased this so its good to go. @iwilltry42 do you want to take this for a spin before I merge it?

@iwilltry42
Copy link
Contributor Author

Hi @donny-dont , thanks for the review! Unfortunately since my project became independent, it's no longer using DroneCI, so I cannot easily test it there.

- bump GitHub dep to v35.3.0 as this was introduced in v35.1.0
- add new string flag "discussion-category" (+ env vars)
- update release: only set category, if not set
- NOTE: Currently it looks like release drafts with discussion category
set do not "persist" the category when being released, so when
publishing a release draft, without explicitly setting the category, it
won't have a release discussion. Nothing we can do here I suppose.

Signed-off-by: Thorsten Klein <[email protected]>
@donny-dont donny-dont force-pushed the feature/release-discussions branch from d98745d to f50cd0c Compare May 10, 2022 21:17
@donny-dont donny-dont merged commit ce75483 into drone-plugins:master May 10, 2022
@iwilltry42 iwilltry42 deleted the feature/release-discussions branch May 11, 2022 05:31
donny-dont added a commit that referenced this pull request May 12, 2022
donny-dont added a commit that referenced this pull request May 12, 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.

2 participants