Skip to content
This repository was archived by the owner on Aug 15, 2019. It is now read-only.

Conversation

@ivikash
Copy link
Contributor

@ivikash ivikash commented Sep 4, 2017

Idea folders are auto generated by WebStorm.


This change is Reviewable

@nsthorat
Copy link
Contributor

nsthorat commented Sep 5, 2017

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.


.gitignore, line 10 at r1 (raw file):

npm-debug.log
.DS_Store
.idea*

.idea is a directory right? to keep this in sync with the format above, can you do .idea/?


Comments from Reviewable

idea folders are auto generated by WebStorm
@ivikash
Copy link
Contributor Author

ivikash commented Sep 6, 2017

@nsthorat done.

@nsthorat
Copy link
Contributor

nsthorat commented Sep 6, 2017

:lgtm_strong:


Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.


Comments from Reviewable

@nsthorat nsthorat merged commit 34d7c8c into tensorflow:master Sep 6, 2017
mnottheone pushed a commit to mnottheone/deeplearnjs that referenced this pull request Dec 1, 2018
* chore(gitignore): add idea folders to gitignore

idea folders are auto generated by WebStorm
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