-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add TypeScript template #62
Conversation
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.
Looks great! Just some minor comments and questions.
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.
Looks great! Just some minor questions / comments.
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.
Removing extra things as discussed.
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.
Almost there! Looking pretty good. Just some minor comments / questions.
src/initStarter.js
Outdated
} | ||
} | ||
}; | ||
|
||
const writeChangeLog = async (rootPath) => { |
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.
I actually think we no longer need to write the CHANGELOG file. This used to be because standard-version
did not write the first version of the file, but now I think they do. Can you confirm / remove, please?
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.
Yes, there is no need to create the changelog file anymore. It is done automatically when doing the first release.
As a side note, the package standard-version
is marked as deprecated since this summer (lack of maintainers i think).
But for the moment it is fine i think. And there is a fork that proposes itself as a successor, see this issue
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.
🎉
This PR adds the typescript template to the cli and adapts it to the new Graasp.
We will release a new version (0.5.0) or maybe a major bump.
closes #61