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(build): use exec plugin for github asset #332

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

svnv
Copy link
Member

@svnv svnv commented Feb 26, 2024

Fixes #331
Ref. #330
Rel. semantic-release/npm#535
Rel. https://github.com/Dintero/Dintero.Checkout.Web.SDK/actions/runs/8045967660/job/21972265552

The previous config change did not work as intended. The npm plugin for semantic release seems to be running twice and the .tgz is not present in the dist folder, and from the logs it does not even look like the package created contains the actual build files either. Instead of trying to make sense of the npm plugin we try using the exec plugin to run npm pack before the github plugin creates is release.

@svnv svnv requested a review from andersem February 26, 2024 12:26
@svnv svnv self-assigned this Feb 26, 2024
@svnv svnv requested a review from frodeaa as a code owner February 26, 2024 12:26
Copy link
Contributor

@andersem andersem left a comment

Choose a reason for hiding this comment

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

🤞

@svnv svnv force-pushed the build-github-release-include-assets-fix branch from a97f042 to 8495a04 Compare February 27, 2024 08:39
@svnv svnv merged commit ecffde2 into master Feb 27, 2024
1 check passed
@svnv svnv deleted the build-github-release-include-assets-fix branch February 27, 2024 08:45
svnv added a commit that referenced this pull request Feb 28, 2024
Ref. https://github.com/Dintero/Dintero.Checkout.Web.SDK/actions/runs/8061936439
Rel. #332
Rel. #332 (review)

Add new dependency used by build. Also updates build actions from version `@3` to `@4` to handle warnings:
> Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Copy link

🎉 This PR is included in version 0.7.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.

.tgz not a part of github release build
2 participants