Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

gitignore all npm-debug logs #3058

Merged
merged 3 commits into from
May 10, 2017
Merged

gitignore all npm-debug logs #3058

merged 3 commits into from
May 10, 2017

Conversation

jmendiara
Copy link
Contributor

Lot of npm-debugs files can be generated in devs computers, each one with a number suffix
ex:

npm-debug.log.213131231
npm-debug.log.213131231
npm-debug.log.213131231

Adding a wildcard prevents for accidental commits

Official and default .gitignore for node in github.com: https://github.com/github/gitignore/blob/master/Node.gitignore#L4

Lot of npm-debugs files can be generated in devs computers, each one with a number suffix
ex:
```sh
npm-debug.log.213131231
npm-debug.log.213131231
npm-debug.log.213131231
```

Adding a wildcard prevents for accidental commits

_Official and default .gitignore for node in github.com: https://github.com/github/gitignore/blob/master/Node.gitignore#L4_
@webron webron requested a review from shockey May 10, 2017 00:10
@webron
Copy link
Contributor

webron commented May 10, 2017

@shockey if makes sense, please apply to projects downstream as well.

@shockey shockey self-assigned this May 10, 2017
@shockey
Copy link
Contributor

shockey commented May 10, 2017

This looks good, thanks for contribution @jmendiara!

@webron, I'll make the changes to JS and Editor as well.

@shockey
Copy link
Contributor

shockey commented May 10, 2017

@shockey shockey merged commit e2003ae into swagger-api:master May 10, 2017
@jmendiara jmendiara deleted the fix/gitignore branch May 10, 2017 21:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants