Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Discord Release Notification #2531

Merged
merged 2 commits into from
Dec 6, 2023
Merged

Add Discord Release Notification #2531

merged 2 commits into from
Dec 6, 2023

Conversation

rhullah
Copy link
Contributor

@rhullah rhullah commented Dec 6, 2023

Add an on-publish step to send a Discord notification when a release is published.

Additionally:

  • removed old workflow names
  • format the args a little nicer
  • remove the redundant expression syntax on if:

I assume this will go to the #ci channel in Discord. Felt like #releases would make more sense, but I'm assuming that's just official Minecraft releases and integrated via Discord.

Add an on-publish step to send a Discord notification when a release is published.

Additionally format the args a little nicer and remove the redundant expression syntax on `if:`.
Copy link
Owner

@itzg itzg left a comment

Choose a reason for hiding this comment

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

Looks great, thanks! Yes, in hindsight the releases channel would have been set aside for image releases and I suppose is could be arranged as such. Let's keep it all in the ci channel until it bugs us and can split things out.

@itzg itzg merged commit 66ed0ed into itzg:master Dec 6, 2023
4 checks passed
@itzg
Copy link
Owner

itzg commented Dec 6, 2023

That's a little annoying that Discord created embeds for the linked PRs. I wonder if surrounding them in angle brackets would suppress that?

https://discord.com/channels/660567679458869252/927361063572365352/1181809421060423770

Not a big deal though.

@itzg
Copy link
Owner

itzg commented Dec 6, 2023

...hmm, surrounding in angle brackets is non-trivial since the GitHub action is just outputting the release notes verbatim.

@rhullah
Copy link
Contributor Author

rhullah commented Dec 6, 2023

Man...you just said everything I was typing up! 😆 Yeah, only options I can think of would be to either tweak the release body to surround the links in <>, or see if that Action has any sort of option to tweak the URLs itself. Do you have any thoughts?

@rhullah
Copy link
Contributor Author

rhullah commented Dec 6, 2023

Hmm, also noticed the HTML comment rendered in the Discord message too. Which means that'd need to be stripped out. I'm playing with something.

@itzg
Copy link
Owner

itzg commented Dec 6, 2023

Yeah, I'm actually hoping that GitHub action or an alternate one has an option to escape links in the body or disable embeds when posting. I vaguely remember that's an API option.

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