-
Notifications
You must be signed in to change notification settings - Fork 39
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
Replace cargo generate #63
Conversation
d3eef0a
to
4d0492f
Compare
@emidev98 would you mind running the eslint fix on both code new and the new command? Lots of issues with the style. |
@octalmage I created this issue to modify all staying in only one PR when we have less PRS open #69. Does it make more sense? Since we have many linings that should be fixed |
It makes the code harder to review, might as well start fixing our code moving forward! |
Makes totally sense it looked horrible, I have fixed all of them minus https://github.com/terra-money/terrain/security/code-scanning/678, which if I am not mistaking we are addressing it in another branch. Let me know @octalmage 👍 |
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.
LGTM
* feat: merge 'code' and 'contract' * feat: terra v2 compatibility (#76)
aed8534
to
9cb7ebb
Compare
This PR will replace cargo generate with template-scaffolding which is a native JavaScript solution to clone templates and replace the tags from the template with necessary values send as config object.