-
-
Notifications
You must be signed in to change notification settings - Fork 89
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 'helper' blueprint #240
Conversation
This PR also adds a way to replace arbitrary properties in the blueprint content when calling `buildProperties` in the emberCLI service.
|
||
return { | ||
filePath: blueprint.filePath, | ||
content: blueprints[blueprint.blueprint].replace(/<\%\=(.*)\%\>/gi,'') | ||
content: content.replace(/<\%\=(.*)\%\>/gi,'') | ||
}; | ||
} | ||
}, |
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 think this method should get a unit 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.
Good catch
@Gaurav0 I added the unit test. Let me know if you need anything else. |
|
||
fileProperties = this.get('emberCli').buildProperties(type, { | ||
camelizedModuleName: name | ||
}); |
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 could see the addFile
method being more like this in the future.
ee203be
to
328d497
Compare
See Travis CI Build. It thinks you are missing a semicolon. |
328d497
to
81014a3
Compare
81014a3
to
377c1b9
Compare
👍 |
This PR also adds a way to replace arbitrary properties in the blueprint
content when calling
buildProperties
in the emberCLI service.Closes #191
Thanks @SaladFork for your Services PR, which I used to bounce off of.