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: postinstall script #32

Merged
merged 5 commits into from
Apr 15, 2024
Merged

fix: postinstall script #32

merged 5 commits into from
Apr 15, 2024

Conversation

sgammon
Copy link
Owner

@sgammon sgammon commented Mar 31, 2024

Summary

Postinstall script is apparently transitive. We only need it for Deno, so this change drops it and issues a new release, at version 1.0.4.

Changelog

  • fix: postinstall script
  • fix: drop distcheck (it's broken)
  • chore: version bump → 1.0.4
  • chore: update commander.js to develop
  • chore: update all node deps

Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
@sgammon sgammon added the bug Something isn't working label Mar 31, 2024
@sgammon sgammon self-assigned this Mar 31, 2024
Adopt changes from tj/commander.js#2170

Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Copy link

sonarcloud bot commented Apr 15, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@sgammon sgammon requested a review from darvld April 15, 2024 16:46
Copy link
Collaborator

@darvld darvld left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@sgammon sgammon merged commit 746c387 into main Apr 15, 2024
29 checks passed
@sgammon sgammon deleted the fix/postinstall branch April 15, 2024 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants