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

C#: Use custom project setting for C# project files name #64460

Merged
merged 1 commit into from
Aug 22, 2022

Conversation

raulsntos
Copy link
Member

Backports 9c69862 to 3.x
Fixes #27030

An important difference from the master version is I renamed the settings to be more consistent with how we name C# settings in 3.x:

  • dotnet/project/assembly_namemono/project/assembly_name
  • dotnet/project/solution_directorymono/project/solution_directory
  • dotnet/project/c#_project_directorymono/project/c#_project_directory

Created as a draft since this is not in master yet (pending the merge of the dotnet6 branch).

The setting is initially assigned the name of the Godot project,
but it's kept freezed to prevent issues when renaming the Godot
project.

The user can always rename the C# project and solution manually and
change the setting to the new name.

Co-authored-by: Ignacio Roldán Etcheverry <ignalfonsore@gmail.com>
@neikeq
Copy link
Contributor

neikeq commented Aug 17, 2022

Created as a draft since this is not in master yet (pending the merge of the dotnet6 branch).

This doesn't need to wait for that one to be merged.

@raulsntos
Copy link
Member Author

Fair enough, but we usually want changes to be merged in master first and I wasn't sure if that code may change before it gets merged.

@raulsntos raulsntos marked this pull request as ready for review August 17, 2022 18:51
@raulsntos raulsntos requested a review from a team as a code owner August 17, 2022 18:51
@akien-mga akien-mga merged commit 67cc142 into godotengine:3.x Aug 22, 2022
@akien-mga akien-mga modified the milestones: 3.x, 3.5, 3.6 Aug 22, 2022
@akien-mga akien-mga added the cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release label Aug 22, 2022
@raulsntos raulsntos deleted the mono_assembly_name-3.x branch August 22, 2022 21:09
@akien-mga
Copy link
Member

Cherry-picked for 3.5.1.

@akien-mga akien-mga removed the cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release label Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants