Skip to content

First addition for NTVS#1163

Merged
arcresu merged 1 commit intogithub:masterfrom
ma499:nodejs-visualstudio
Jan 7, 2015
Merged

First addition for NTVS#1163
arcresu merged 1 commit intogithub:masterfrom
ma499:nodejs-visualstudio

Conversation

@ma499
Copy link
Copy Markdown
Contributor

@ma499 ma499 commented Aug 8, 2014

First exclusion for Node.js tools for Visual Studio 2013.

@arcresu
Copy link
Copy Markdown
Contributor

arcresu commented Aug 13, 2014

Thanks for this! Can you explain a little more about what this file is for (maybe link to some documentation)? The VS template is already pretty long so it's nice to be sure that any new additions will be widely useful and sensible.

@arcresu
Copy link
Copy Markdown
Contributor

arcresu commented Sep 1, 2014

Thanks, but I'll close this for now. Feel free to reopen with more information.

@arcresu arcresu closed this Sep 1, 2014
@ma499
Copy link
Copy Markdown
Contributor Author

ma499 commented Jan 2, 2015

Sorry I missed this earlier. .ntvs_analysis.dat basically caches the IntelliSense and JavaScript code analysis output. It can get very large and like Visual Sutdio user preferences (*.suo, *.ide) shouldn't really be checked into source. See this discussion with responses from the Node.js tools team

Would you like to have this information in the template gitignore? Or did you just want to sanity-check the correctness of putting this file in?

@ma499 ma499 changed the title First addition for NTVS. First addition for NTVS Jan 2, 2015
@ma499
Copy link
Copy Markdown
Contributor Author

ma499 commented Jan 2, 2015

I'm not sure how to re-open the pull request. Do I just create a new pull request from my fork?

@arcresu arcresu reopened this Jan 2, 2015
@arcresu
Copy link
Copy Markdown
Contributor

arcresu commented Jan 7, 2015

No need to change the PR, it's just nice to have some confirmation in the discussion for future reference. Thanks for linking to that discussion ✨

arcresu added a commit that referenced this pull request Jan 7, 2015
Ignore analysis files from Node.js tools for Visual Studio 2013
@arcresu arcresu merged commit 95a6bda into github:master Jan 7, 2015
u9E9F pushed a commit to u9E9F/gitignore that referenced this pull request Jul 8, 2017
Ignore analysis files from Node.js tools for Visual Studio 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants