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

Create .gitignore when execute init command #957

Merged
merged 4 commits into from Mar 9, 2018

Conversation

sottar
Copy link
Contributor

@sottar sottar commented Feb 5, 2018

PR for #953

Summary

Automatically generate a .gitignore file when execute init command and select element or application.

  • CHANGELOG.md has been updated

Copy link
Contributor

@FredKSchott FredKSchott left a comment

Choose a reason for hiding this comment

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

Thanks, great catch!

Just 1 small comment otherwise looks good

gulpfile.js Outdated
@@ -37,7 +37,8 @@ gulp.task('compile', () => {
let tsReporter = typescript.reporter.defaultReporter();
return mergeStream(
tsProject.src().pipe(tsProject(tsReporter)),
gulp.src(['src/**/*', '!src/**/*.ts']))
gulp.src(['src/**/*', '!src/**/*.ts']),
gulp.src(['src/**/.gitignore']))
Copy link
Contributor

Choose a reason for hiding this comment

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

This like should be unnecessary, no need to compile .gitignore with typescript

@sottar
Copy link
Contributor Author

sottar commented Feb 6, 2018

@FredKSchott Thanks for your reviews !
I added copy task in gulpfile. It just copy .gitignore to lib directory.

@stramel
Copy link
Contributor

stramel commented Feb 6, 2018

Did yeoman make it to where it doesn't ignore hidden files? I remember looking at this a while back and thought it would ignore hidden files.

@sottar have you checked this?

@sottar
Copy link
Contributor Author

sottar commented Feb 6, 2018

@stramel
I confirmed that a .gitignore file was generated in my branch.

@justinfagnani
Copy link
Contributor

@FredKSchott I restarted the tests and they pass... lgty?

@stramel
Copy link
Contributor

stramel commented Mar 9, 2018

Fixes #531 as well

@justinfagnani justinfagnani merged commit 76bb6b0 into Polymer:master Mar 9, 2018
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.

None yet

4 participants