-
Notifications
You must be signed in to change notification settings - Fork 94
added new arg base64file to support loading base64 blob from file int… #938
Conversation
commands/call.js
Outdated
parsedArgs = Buffer.from(fileargs, 'base64'); | ||
console.log(`Loaded base64 args file, size ${parsedArgs.length}`); | ||
} catch (e) { | ||
console.log('Could not load base64 file!'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this throw so as exit? Currently all errors throw and you end up with a large noisy help message, but there are plans to refactor how errors are handled.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call, changed!
@willemneal looks like the last build failed because RPC 502: https://app.travis-ci.com/github/near/near-cli/builds/246520845#L260 |
@willemneal should i trigger with a commit, or can you manually trigger a rebuild? I think RPC services shipped recently are fixed now. |
@TrevorJTClarke I wish I could, but I'm not a collaborator. So you could maybe do a push again or a small change and push, then revert and force push? |
@willemneal nevermind, same RPC 502... Some config or something got changed. No idea how to help get this across the line. Super needed in timely manner - Auditors starting in 8 days need this |
@TrevorJTClarke You can depend on your version of near-cli. I had to do that while waiting for this PR: #903, which has been released and could also be used to solve your issue. |
This has been stalled for more than 2 years, I will proceed to close it and we can re-open it if the community requires it. With that being said, I think it is quite simple to do this in the command line, we actually show how to do it when relevant in our docs: https://docs.near.org/tutorials/examples/factory#update-the-stored-contract |
…o CLI args
This PR fixes #937
Directly impacting any CLI args on linux