-
Notifications
You must be signed in to change notification settings - Fork 7
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
Zulu coda test and refactor #4
Conversation
… create all templates will have to be refactored to by using async each series in order to be able to add callback which will be called once everything has been completed! ✊
54c55fe
to
776f0a7
Compare
…rydom/Generate-Redux-Code into zulu-coda-test-and-refactor
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.
Dude, i think we should have merged PR after the tests pass (2e19c8a).
then add these other changes to a new PR.
src/generator-utilities.js
Outdated
@@ -141,7 +141,7 @@ function removeComments(s) { | |||
return s.replace(/([\s\S]*?)\/\*[\s\S]*?\*\//g, '$1'); | |||
} | |||
|
|||
function createTemplate(directory, placeholderNames, omitComments) { | |||
function createTemplate(directory, placeholderNames, omitComments, callback) { |
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.
Dude this breaks test
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.
it should be return callback(err, 'code generated from template');
its cool, I fixed the merge issue |
… refactored to use a callback so that it can be test friendly ✊
okay great back to green 👍 |
The one test doesn't clean up after itself. We'll have to fix that first. We'll also have to write some |
Yeah, finally all green 👍 |
…rydom/Generate-Redux-Code into zulu-coda-test-and-refactor
…an error because of the throw statement)
…rydom/Generate-Redux-Code into zulu-coda-test-and-refactor
… called after writing to file ✊
ok cool build is green, lets get this ONE merged 👍 |
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
you gonna |
To Test PR clone this REPO: https://github.com/zulucoda/Generate-React-Code-PR-Test