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

Update gdscript_styleguide.rst - clarified sentence and grammar #9110

Merged
merged 2 commits into from
Mar 21, 2024

Conversation

444B
Copy link
Contributor

@444B 444B commented Mar 17, 2024

  • Clarified the meaning of "One statement per line"
  • Removed unclear sentence with incorrect grammar

Thank you @AThousandShips for your patience :)
Correctly started from master branch and not 4.2

- Clarified the statement on "One statement per line"
- Removed unclear sentence with incorrect grammar
@AThousandShips
Copy link
Member

Thank you for your patience :) this is confusing and complicated and the interface isn't always obvious

@AThousandShips AThousandShips added topic:gdscript area:manual Issues and PRs related to the Manual/Tutorials section of the documentation labels Mar 17, 2024
@AThousandShips AThousandShips requested a review from dalexeev March 17, 2024 12:44
Co-authored-by: A Thousand Ships <96648715+AThousandShips@users.noreply.github.com>
@444B
Copy link
Contributor Author

444B commented Mar 18, 2024

ready to submit with no other changes

@444B
Copy link
Contributor Author

444B commented Mar 21, 2024

@AThousandShips @dalexeev
Is anything else required to merge this?

@dalexeev
Copy link
Member

@444B No, there are no suggestions. Please wait while a maintainer merges this. Thanks!

@mhilbrunner mhilbrunner merged commit 3d44115 into godotengine:master Mar 21, 2024
1 check passed
@mhilbrunner
Copy link
Member

Merged. Thanks and congrats on your first merged contribution!

@444B
Copy link
Contributor Author

444B commented Mar 21, 2024

Thank you!
Hoping to make more contributions and to be a part of the community :)

@444B 444B deleted the patch-3 branch March 23, 2024 09:39
mhilbrunner pushed a commit that referenced this pull request Jul 24, 2024
* Update gdscript_styleguide.rst - clarified sentence and grammar

- Clarified the statement on "One statement per line"
- Removed unclear sentence with incorrect grammar

---------

Co-authored-by: A Thousand Ships <96648715+AThousandShips@users.noreply.github.com>
(cherry picked from commit 3d44115)
@mhilbrunner
Copy link
Member

Cherry-picked to 4.2 in #9648.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:gdscript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants