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 a typo in the String.to_float description #78989

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

Amir-Rasteg
Copy link
Contributor

Fix very minor typo in String class documentation, where an example for "to_float" has a "to_int" within it, despite also returning a float. Given these factors, this is likely a typo

@Amir-Rasteg Amir-Rasteg requested a review from a team as a code owner July 3, 2023 16:14
@YuriSizov YuriSizov changed the title Update String.xml Fix a typo in the String.to_float description Jul 3, 2023
@YuriSizov
Copy link
Contributor

Thanks for opening a pull request! Please amend your commit message to state with a bit more detail what your commit does. E.g. "Fix a typo in the String.to_float description", like I just did with the PR title.

@YuriSizov YuriSizov added this to the 4.2 milestone Jul 3, 2023
@YuriSizov YuriSizov added cherrypick:4.0 cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release labels Jul 3, 2023
@akien-mga
Copy link
Member

akien-mga commented Jul 6, 2023

Force pushed a rebase to remove the duplicate and merge commits.
Also fixed the alignment of the comments to match the previous style and other similar examples.

@akien-mga akien-mga merged commit ecca4e5 into godotengine:master Jul 7, 2023
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

@YuriSizov YuriSizov removed the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Jul 10, 2023
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.1.1.

@YuriSizov
Copy link
Contributor

Cherry-picked for 4.0.4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants