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

Fix debugger warnings about unused identifiers #177

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Jayman2000
Copy link
Contributor

See individual commits for details.

This commit is just a bunch of changes that Godot 3.5.2 will
automatically make when editing projects that were created with an older
version of Godot.
If the identifier was used by code that was commented out, then I just
commented out the line that declared the identifier. If the identifier
wasn’t even used by code that was commented out, then I just deleted the
line that declared the identifier.
@abitrolly
Copy link

@Jayman2000 how did you run newer godot over all files to update them? If I just open and close project in editor, only ./project.godot is updated. If I toggle label visibility and move viewport, then scenes/title.tscn and styles/theme.tres updated too. But all other files from 427ebb3 like cards/add.svg.import etc. are left intact.

@Jayman2000
Copy link
Contributor Author

Jayman2000 commented Dec 13, 2023

When I created that commit, I think that I had just opened and saved each of the Resources manually. I just made a script that you can use to update all of the files automatically. Here’s how you can use that script:

  1. Download update_files_for_new_version_of_godot.gd.
  2. Put it in the project folder.
  3. Open that script in Godot.
  4. In the script editor, right click on the script’s name, and click “Run”.
  5. Select Scene > Save All Scenes

Note that if you have “Trim Trailing Whitespace On Save” enabled in the Godot’s Editor Settings, then that script will also remove any trailing whitespace.

@Jayman2000 Jayman2000 closed this Dec 13, 2023
@Jayman2000 Jayman2000 reopened this Dec 13, 2023
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.

None yet

2 participants