X Tutup
The Wayback Machine - https://web.archive.org/web/20210210103829/https://github.com/github/gitignore/pull/3540
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

VisualStudio: add "out" folder to build results #3540

Open
wants to merge 1 commit into
base: master
from

Conversation

@jumattos
Copy link

@jumattos jumattos commented Oct 1, 2020

Reasons for making this change:

Out is a common folder for output that should not be committed. It is also the default output folder for CMake Projects in Visual Studio. See supporting feedback ticket at Developer Community.

Links to documentation supporting these rule changes:

See documentation for VS 2019: CMake projects in Visual Studio - IDE Integration.

Click the Show All Files button at the top of Solution Explorer to see all the CMake-generated output in the out/build/ folders.

@53jk1
53jk1 approved these changes Oct 23, 2020
@jumattos
Copy link
Author

@jumattos jumattos commented Oct 23, 2020

@shiftkey Can you take a look at this PR? Thanks!

@ghost
Copy link

@ghost ghost commented Nov 13, 2020

@shiftkey +1 to merge this PR.

None of the .gitignore templates actually cover this, and as it is the default output folder for CMake projects in Visual Studio, that makes sense to add it for the Visual Studio template.

@ghost
Copy link

@ghost ghost commented Nov 13, 2020

Linked to my PR #3576, the following documentation also points out the "out" folder.

"remoteCopySourcesExclusionList": [ ".vs", ".git", "out" ],

@mcleary
Copy link

@mcleary mcleary commented Dec 3, 2020

This would be really useful as I always need to add out/ to the my .gitignore files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
X Tutup