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 ignore for .vsconfig #4530

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add ignore for .vsconfig #4530

wants to merge 1 commit into from

Conversation

BibbitM
Copy link

@BibbitM BibbitM commented Oct 30, 2024

The engine generates the file during the build.

Reasons for making this change:

I'm a game developer working professionally on game projects developed in Unreal Engine.
Also, I try to work on different UE projects and plugins in my spare time.

The Unreal Engine generates the .vsconfig file during the build process.
I don't want to update or ignore the file when working on different projects.

The file is generated each time you regenerate the solution file, and you should not add it to your VCS.

Links to documentation supporting these rule changes:

".vsconfig" is mentioned in the UE documentation - https://dev.epicgames.com/documentation/en-us/unreal-engine/build-configuration-for-unreal-engine

$ bVsConfigFile : Whether to write a .vsconfig file next to the sln to suggest components to install.

https://github.com/EpicGames/UnrealEngine/blob/40eea367040d50aadd9f030ed5909fc890c159c2/Engine/Source/Programs/UnrealBuildTool/ProjectFiles/VisualStudio/VCProjectFileGenerator.cs
The file is generated in the function UnrealBuildTool.VCProjectFileGenerator.WritePrimaryProjectFile the file is generated
The function UnrealBuildTool.VCProjectFileGenerator.CleanProjectFiles shows that the file is removed together with ".sln" and ".suo" files.

The engine generates the file during the build.
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.

1 participant