Skip to content

Conversation

@JonasSchubert
Copy link
Contributor

@JonasSchubert JonasSchubert commented Mar 22, 2022

First of all thanks to @fgreinacher for this nice feature.
We are using it already and I stumpled upon two issues:

  • the 📦 and 🚀 icon were not proberly displayed as they were not separated
  • the URL to the release did not work because of %2F in the URI; this is why I added the decodeURIComponent

Fixes #344

@JonasSchubert JonasSchubert changed the title Fix/success message fix: display default icons and fix URL in success message Mar 22, 2022
@fgreinacher
Copy link
Contributor

Thanks a lot @JonasSchubert! Left a suggestion for you :)

@fgreinacher
Copy link
Contributor

@travi Do you have any idea why the tests are not starting anymore?

@JonasSchubert
Copy link
Contributor Author

JonasSchubert commented Mar 24, 2022

First-time contributors need a maintainer to approve running workflows.

😄
Has to be approved by another reviewer to run the tests

Copy link
Contributor

@fgreinacher fgreinacher left a comment

Choose a reason for hiding this comment

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

LGTM!

@fgreinacher fgreinacher merged commit e6fa727 into semantic-release:master Mar 24, 2022
@github-actions
Copy link

🎉 This PR is included in version 8.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

hvanoch pushed a commit to hvanoch/semantic-release-gitlab that referenced this pull request Apr 14, 2022
…elease#342)

Fixes semantic-release#344

Co-authored-by: Jonas Schubert <jonas.schubert@siemens.com>
Co-authored-by: Florian Greinacher <florian@greinacher.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

successComment has bad formatted emojis

3 participants