Skip to content
This repository was archived by the owner on Oct 31, 2024. It is now read-only.

fix: fixing release notification #423

Merged
merged 2 commits into from
Jan 5, 2024
Merged

Conversation

Freyskeyd
Copy link
Member

@Freyskeyd Freyskeyd commented Jan 3, 2024

Description

This PR aims to fix the redundant notifications emitted when creating a release.

PR Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added or updated tests that comprehensively prove my change is effective or that my feature works

@Freyskeyd Freyskeyd force-pushed the chore/fixing-release-notification branch from 10c618a to 821e2a7 Compare January 3, 2024 17:28
@Freyskeyd Freyskeyd marked this pull request as ready for review January 3, 2024 17:29
@Freyskeyd Freyskeyd requested a review from a team as a code owner January 3, 2024 17:29
@Freyskeyd Freyskeyd requested review from dvdplm and atanmarko January 3, 2024 17:29
@Freyskeyd Freyskeyd force-pushed the chore/fixing-release-notification branch 2 times, most recently from 96650a5 to 4ec927f Compare January 3, 2024 18:10
Signed-off-by: Simon Paitrault <simon.paitrault@gmail.com>
Signed-off-by: Simon Paitrault <simon.paitrault@gmail.com>
@Freyskeyd Freyskeyd force-pushed the chore/fixing-release-notification branch from 4ec927f to f7bb7ea Compare January 4, 2024 09:19
@dvdplm
Copy link
Contributor

dvdplm commented Jan 5, 2024

Maybe add a comment saying "# No notifications for pre-releases" or something?

@Freyskeyd Freyskeyd merged commit 7503fa7 into main Jan 5, 2024
@Freyskeyd Freyskeyd deleted the chore/fixing-release-notification branch January 5, 2024 18:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants