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

Add .vs folder to .gitignore #360

Merged
merged 3 commits into from
Aug 14, 2023
Merged

Add .vs folder to .gitignore #360

merged 3 commits into from
Aug 14, 2023

Conversation

Mrxx99
Copy link
Contributor

@Mrxx99 Mrxx99 commented Aug 13, 2023

This adds the .vs folder of Visual Studio to .gitignore. Without this it is not possible to switch branches inside Visual Studio as there are always changes (e.g. when only opening a file), but these are locked by Visual Studio.

I also added the .gitignore file to the "Solution Items" folder of the Solution

@Tyrrrz Tyrrrz added the enhancement New feature or request label Aug 14, 2023
.gitignore Outdated Show resolved Hide resolved
@Tyrrrz
Copy link
Owner

Tyrrrz commented Aug 14, 2023

Thanks! I discarded changes to YoutubeDownloader.sln because it seems to involve things irrelevant to updating the .gitignore file.

@Tyrrrz Tyrrrz changed the title Add .vs folder to .gitignore Add .vs folder to .gitignore Aug 14, 2023
@Tyrrrz Tyrrrz merged commit 6e59f90 into Tyrrrz:master Aug 14, 2023
3 checks passed
@Mrxx99
Copy link
Contributor Author

Mrxx99 commented Aug 14, 2023

The changes in sln was adding it to the Solution items folder to be able to edit easily in Visual Studio

@Tyrrrz
Copy link
Owner

Tyrrrz commented Aug 14, 2023

The changes in sln was adding it to the Solution items folder to be able to edit easily in Visual Studio

It's fine, I prefer to keep it outside the solution anyway.

@Mrxx99 Mrxx99 deleted the fix-gitignore branch August 14, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants