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.

Sign up for GitHub

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

Jump to bottom

style: 🎨 added and removed whitespace #292

Merged
merged 3 commits into from
Jun 21, 2023
Merged

style: 🎨 added and removed whitespace #292

merged 3 commits into from
Jun 21, 2023

Conversation

KANAjetzt
Copy link
Member

@KANAjetzt KANAjetzt commented Jun 21, 2023 •
edited
Loading

  • Added a missing empty line between functions
  • Automatically removed whitespace via Editor Settings -> Text Editor -> Files -> Trim Trailing Whitespace On Save
  • Automatically added empty line at the bottom of the file in mod.gd
  • added space to params and return types
  • Updated the path comment in ModLoaderStore for
    • user profiles file user://mods.json -> user://mod_user_profiles.json
    • cache file user://ModLoaderCache.json -> user://mod_loader_cache.json

@KANAjetzt KANAjetzt added this to the v6.0.1 milestone Jun 21, 2023
@KANAjetzt KANAjetzt requested a review from a team June 21, 2023 07:04
@KANAjetzt KANAjetzt self-assigned this Jun 21, 2023
Qubus0
Qubus0 approved these changes Jun 21, 2023
View reviewed changes
addons/mod_loader/internal/script_extension.gd Outdated Show resolved Hide resolved
@KANAjetzt KANAjetzt added this pull request to the merge queue Jun 21, 2023
Merged via the queue into GodotModding:development with commit 4a7b512 Jun 21, 2023
@KANAjetzt KANAjetzt deleted the style_white-space branch June 21, 2023 07:49
@KANAjetzt KANAjetzt mentioned this pull request Jun 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Qubus0 Qubus0 Qubus0 approved these changes

Assignees

@KANAjetzt KANAjetzt

Labels
style
Projects
None yet
Milestone
v6.0.1
Development

Successfully merging this pull request may close these issues.
2 participants
@KANAjetzt @Qubus0