Skip to content

style: resolve type and style requests from #562 #566

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

Merged
merged 5 commits into from
Jul 8, 2025

Conversation

ZackeryRSmith
Copy link
Member

@ZackeryRSmith ZackeryRSmith commented Jun 28, 2025

A tiny PR (+10-10) which applies the changes outlined by @KANAjetzt in #562

Auto merge was turned on for the PR, and the changes of #562 were merged before I could resolve the nit picks. This PR resolves those.

@KANAjetzt KANAjetzt changed the title Resolve nit picks from #562 style: Resolve nit picks from #562 Jun 29, 2025
Copy link
Member

@KANAjetzt KANAjetzt left a comment

Choose a reason for hiding this comment

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

One other small thing, it is helpful to create a separate feature branch for your PRs, it makes it easier for the reviewers to check the branch out. If the PR is using the dev branch directly, the reviewer will most likely have to delete their dev branch first.

Thanks! 😁

@KANAjetzt KANAjetzt added this to the 4.x - 7.x milestone Jun 29, 2025
@ZackeryRSmith
Copy link
Member Author

One other small thing, it is helpful to create a separate feature branch for your PRs, it makes it easier for the reviewers to check the branch out. If the PR is using the dev branch directly, the reviewer will most likely have to delete their dev branch first.

Thanks! 😁

Sorry about any issue my PRing shenanigans caused, I'll keep this in mind from now on!

@ZackeryRSmith ZackeryRSmith requested a review from KANAjetzt July 4, 2025 20:27
@KANAjetzt KANAjetzt changed the title style: Resolve nit picks from #562 style: resolve type and style requests from #562 Jul 8, 2025
@KANAjetzt KANAjetzt merged commit f2d9954 into GodotModding:4.x-dev Jul 8, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants