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

feat(cli): new subcommands #283

Merged
merged 2 commits into from
Nov 14, 2022

Conversation

waynenilsen
Copy link
Contributor

@waynenilsen waynenilsen commented Nov 3, 2022

Problem Statement

We may want to use parts of this build system with parts of our own build system. I may have use of using my own build script to call rollup and tsc.

Solution

To start this conversation I suggest we make new subcommands for validating and doing other things step-by-step. This is somewhat designed as a sketch to gather feedback and see if you are interested in this kind of work.

How has this been tested?

ok I tested out with my contract, this should work fine, will take another look at unit tests


related issue

@waynenilsen
Copy link
Contributor Author

waynenilsen commented Nov 3, 2022

another option is to export and/or move the funcitons called in buildCom such as createHeaderFileWithQjsc

previously - this was marked as a draft - but the point of this pr is to have a discussion so i moved it back to open state

@waynenilsen waynenilsen marked this pull request as draft November 4, 2022 13:01
@waynenilsen waynenilsen marked this pull request as ready for review November 4, 2022 16:11
@volovyks
Copy link
Collaborator

volovyks commented Nov 8, 2022

It's a great idea @waynenilsen ! I think we want to have a command for all the steps of the build process + one command that will run the pipeline (build). It will help a lot during contract development and library development. Do you want to extend this PR with all the commands?

@waynenilsen
Copy link
Contributor Author

yes i can take care of it

@waynenilsen waynenilsen force-pushed the wayne/extract-command-example branch 4 times, most recently from 1aaac6f to de9a81f Compare November 10, 2022 19:12
@waynenilsen waynenilsen changed the title feat(cli): new subcommand(s) - sketch feat(cli): new subcommands Nov 10, 2022
### Problem Statement

We may want to use parts of this build system with parts of our own build system. I may have use of using my own build script to call rollup and tsc.

### Solution

To start this conversation I suggest we make new subcommands for validating and doing other things step-by-step. This is somewhat designed as a sketch to gather feedback and see if you are interested in this kind of work.

### How has this been tested?

starting to manually run some of the tests that run in CI noticed a bug

something going on with CI build was not working for some reason
---

[related issue](near#280)
Copy link
Member

@ailisp ailisp left a comment

Choose a reason for hiding this comment

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

Looks good to me! Yes this is a great help to run one specific step of the build process. Especially for advanced users and near-sdk-js maintainers

@volovyks volovyks merged commit 3f170ec into near:develop Nov 14, 2022
@waynenilsen waynenilsen deleted the wayne/extract-command-example branch November 14, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants