Skip to content
This repository was archived by the owner on Dec 7, 2021. It is now read-only.

add new polymer 2.0 element and application templates#554

Merged
FredKSchott merged 3 commits intomasterfrom
init-new-templates
Feb 4, 2017
Merged

add new polymer 2.0 element and application templates#554
FredKSchott merged 3 commits intomasterfrom
init-new-templates

Conversation

@FredKSchott
Copy link
Copy Markdown
Contributor

  • CHANGELOG.md has been updated

The diff is bigger than it looks, there are a lot of files moved around and created to add the two new Yeoman templates. You can find most changes in src/init/init.ts.

Copy link
Copy Markdown
Contributor

@justinfagnani justinfagnani left a comment

Choose a reason for hiding this comment

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

💯


export class ApplicationGenerator extends Generator {
props: any;
export function createApplicationGenerator(templateName: string):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

doc that the template name is both the display name and where the template files are loaded from


export class ElementGenerator extends Generator {
props: any;
export function createElementGenerator(templateName: string):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also doc this

@FredKSchott
Copy link
Copy Markdown
Contributor Author

Comments addressed, letting CI run and then will merge after

@FredKSchott FredKSchott merged commit b6ffbe2 into master Feb 4, 2017
@FredKSchott FredKSchott deleted the init-new-templates branch February 4, 2017 00:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants