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

fix: Publish artifacts with semantic-release releases #1094

Merged
merged 2 commits into from
Jun 15, 2021
Merged

fix: Publish artifacts with semantic-release releases #1094

merged 2 commits into from
Jun 15, 2021

Conversation

manthey
Copy link
Contributor

@manthey manthey commented Jun 10, 2021

With semantic release, there is a different technique to publish the build artifacts to the github releases. This switches to it.

@manthey manthey requested a review from aashish24 June 10, 2021 13:05
@manthey
Copy link
Contributor Author

manthey commented Jun 10, 2021

@aashish24 If you look at the release page, there is currently a v1.0.3-beta.1 release. You can see that it has 6 build artifacts:
image
Before, it just showed the file names:
image

With semantic-release it is easy to control the names that appear. Maybe it would be better to have something like:
Full JS Distribution (geo.js)

Thoughts?

It looks like this will maintain our manual changes, too.
@manthey
Copy link
Contributor Author

manthey commented Jun 10, 2021

Closes #1095.

@aashish24
Copy link
Member

I think Full JS Distribution (geo.js) will appeal better

@manthey
Copy link
Contributor Author

manthey commented Jun 10, 2021

I agree. I made that change in the second commit on this branch. I think it will also now update the CHANGELOG correctly, too (but I'm not sure how test it other than by merging).

@manthey manthey merged commit 6e0b1dd into master Jun 15, 2021
@manthey manthey deleted the beta branch June 15, 2021 14:04
@github-actions
Copy link

🎉 This PR is included in version 1.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

2 participants