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

[3.x] Document editor settings in the class reference #64896

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Aug 25, 2022

3.x version of #63870.

Marked as draft as #64895 must be merged first for CI to pass.
Edit: No longer draft.

@Calinou Calinou added this to the 3.x milestone Aug 25, 2022
@Calinou Calinou force-pushed the doc-editorsettings-3.x branch from ae9d006 to 918a8d7 Compare August 25, 2022 18:06
@Calinou Calinou force-pushed the doc-editorsettings-3.x branch from 918a8d7 to b84b003 Compare October 3, 2022 22:43
@akien-mga
Copy link
Member

Is this still a draft?

Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

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

I didn't proofread in depth, but should be good to merge if it's ready, and any typos can be fixed up later on.

@Calinou Calinou marked this pull request as ready for review October 4, 2022 17:16
@Calinou Calinou requested a review from a team as a code owner October 4, 2022 17:16
@Calinou
Copy link
Member Author

Calinou commented Oct 4, 2022

Is this still a draft?

I forgot to mark this as ready for review – it should be OK to merge now.

@akien-mga akien-mga merged commit 07f7d6a into godotengine:3.x Oct 5, 2022
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the doc-editorsettings-3.x branch October 5, 2022 16:09
@akien-mga akien-mga changed the title Document editor settings in the class reference [3.x] Document editor settings in the class reference Sep 9, 2024
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.

2 participants