C#: Add note about the class name in instantiate error #70210
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a note about the requirement that a C# class name must match the script filename in which the they are defined to the instantiate error.
This should hopefully make the error more user-friendly and avoid issues such as #66419 in the future.
Feel free to suggest a better message, the goal is to mention the common reasons why the associated class was not found:
In case the user attached a script to a Node and then deleted the script without detaching it from the Node.
It seems common for users to define a class such as
Player
using PascalCase to follow .NET class naming conventions and then name the scriptplayer.cs
because Godot uses snake_case for filenames by default.