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 gdlint errors #691

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

Fix gdlint errors #691

wants to merge 2 commits into from

Conversation

squidt
Copy link
Contributor

@squidt squidt commented Oct 26, 2024

  • IMPORTANT: 'arcade_hoops_ctrl.gd' line 96 had a 'tween.kill' that does not call 'tween.kill()'. It has been removed to preserve identical functionality while removing a glint error. It is unclear if this needed to be called but was missing.
  • Reorder variables
  • Lessen lines over 100
  • Remove trailing whitespace
  • Prepend "_" to unused function parameters

The most important fix would be to 'start_xr.gd' which reorders the static variable to the top. I've seen at least 2 PRs fail gdlint checks due to this file.

Copy link
Contributor

@DigitalN8m4r3 DigitalN8m4r3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch, yeah those whitespaces snuck in cuz i was a pestilence when i made that pr with the audio demo :D, mux just wanted to get it in and make me stop ;)

kidding aside, so far this looks good to me

@squidt
Copy link
Contributor Author

squidt commented Oct 27, 2024

thanks! yea I at first I was just gonna fix the start_xr file then figured I'd do the rest while i was there since it's an accessible contribution

- IMPORTANT: 'arcade_hoops_ctrl.gd' line 96 had a 'tween.kill' that does not call 'tween.kill()' it has been removed to preserve identical functionality while removing a gdlint error. Unknown if this needed to be called but was missing.
- Reorder variables
- Lessen lines over 100
- Remove trailing whitespace
- Prepend "_" to unused function parameters
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.

2 participants