Skip to content
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

Patch 1 #5220

Closed
wants to merge 1 commit into from
Closed

Patch 1 #5220

wants to merge 1 commit into from

Conversation

johnpapa
Copy link
Contributor

@johnpapa johnpapa commented Mar 4, 2017

Added /out-tsc to the .gitignore blueprint. If someone runs tsc -p src/tsconfig.spec.json it outputs to the ../out-tsc folder. These should be ignored.

Also changed https://github.com/angular/angular-cli/blob/master/packages/%40angular/cli/blueprints/ng/files/e2e/tsconfig.e2e.json#L13 to ../out-tsc/e2e so that they all match

cc @filipesilva

fixing due to issue with CLA from #5062

@delasteve
Copy link
Contributor

@johnpapa, it appears your commit message needs to be updated.

@johnpapa
Copy link
Contributor Author

johnpapa commented Mar 6, 2017

@filipesilva let me know what you need me to do here. seems it is not passing

@delasteve
Copy link
Contributor

@johnpapa, you should be able to squash the commit into one, give it a commit message following the guidelines (below), and force push it back up. If you need help on how to do that, let me know, I'd gladly help where I can.

https://github.com/angular/angular-cli/blob/master/CONTRIBUTING.md#commit

@johnpapa
Copy link
Contributor Author

johnpapa commented Mar 7, 2017

squashed

asnowwolf pushed a commit to asnowwolf/angular-cli that referenced this pull request Apr 12, 2017
…to .gitignore

Added /out-tsc to the .gitignore blueprint. If someone runs tsc -p src/tsconfig.spec.json it outputs to the ../out-tsc folder. These should
be ignored.

Also changed
https://github.com/angular/angular-cli/blob/master/packages/%40angular/cli/blueprints/ng/files/e2e/tsconfig.e2e.json#L13 to ../out-tsc/e2e so that they all match.

cc @filipesilva

fixing due to issue with CLA from angular#5062

Close angular#5220
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 11, 2019
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.

4 participants