feat: add @erpc-cloud/cli + publish package via CI#143
Merged
aramalipoor merged 14 commits intoerpc:mainfrom Jan 23, 2025
Merged
feat: add @erpc-cloud/cli + publish package via CI#143aramalipoor merged 14 commits intoerpc:mainfrom
@erpc-cloud/cli + publish package via CI#143aramalipoor merged 14 commits intoerpc:mainfrom
Conversation
aramalipoor
reviewed
Jan 7, 2025
| @@ -0,0 +1,23 @@ | |||
| #!/usr/bin/env node | |||
Member
There was a problem hiding this comment.
Is there a way in NPM to point directly to a binary without the process wrapper? That would be awesome to avoid an extra hop
Contributor
Author
There was a problem hiding this comment.
Should be doable, let me try that.
Contributor
Author
There was a problem hiding this comment.
Ok so not working super well, basically we got a concurrency issue in the install hooks of the script, it's trying to create the bin symlinks before the end of the postInstall script, so when the bin isn't yet downloaded :/
aramalipoor
reviewed
Jan 7, 2025
zekiblue
pushed a commit
to zekiblue/erpc
that referenced
this pull request
May 6, 2025
* feat: startup @erpc-cloud/cli * feat: checksum validation during installation * chore: new ci release flow with npmjs release * chore: remove release please go with dispatch * test: test ci v1 * test: fix ci contain check * fix: Only build docker with typescript/config package * fix: Cleanup generated file before release commit + fix file path * fix: Dont depend on prepare release for the release step * fix: Disable git check on publish * fix: Correct export of cli package * chore: rollback test for PR * chore: remove bin from pr * chore: remove v2 tag on the release workflow
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note: Need to add the
NPM_TOKENthat will be use for publishing in the github action secret.Add a new
@erpc-cloud/clitypescript package that:Add some stuff to the CI:
tygotypedef for config packagecli(checksums and version info)