Skip to content

ClassBuilder, ToVariant/FromVariant and Export explanations. #51

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 2 commits into from
Nov 24, 2021

Conversation

jacobsky
Copy link
Contributor

To resolve Issue #38 I have added some entries in rust-bindings/properties.md
In addition, I added a section that can better explain the ToVariant, FromVariant and Export traits as these are going to end up being really common issues for folks that work heavily with the Godot Editor.

I am opening this as a draft for some initial feedback on the ideas not necessarily the typos, if I have any major misunderstandings in my explanations please let me know so I can update it. Please don't worry about typos for the time being as I plan to go over and edit this again.

@jacobsky jacobsky marked this pull request as ready for review November 3, 2021 14:29
To resolve Issue godot-rust#38 I have added some entries in `rust-bindings/properties.md`
In addition, I added a section that can better explain the `ToVariant`, `FromVariant` and `Export` traits as these are going to end up being really common issues for folks that work heavily with the Godot Editor
@jacobsky
Copy link
Contributor Author

I've gone over and fixed up this PR.

@Bromeon If you have any feedback or nitpicks, please let me know otherwise I'll go ahead and squash/merge it.

@jacobsky jacobsky requested a review from Bromeon November 14, 2021 11:18
Copy link
Member

@Bromeon Bromeon left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the write-up!
I added some comments.

@jacobsky
Copy link
Contributor Author

I added the fixes as requested. Let me know if there's anything else that you'd like to see changed :)

@Bromeon
Copy link
Member

Bromeon commented Nov 24, 2021

Should be good, thanks! 🙂

@jacobsky
Copy link
Contributor Author

Thank you for the reviews! I'll go ahead and squash/merge the changes :)

@jacobsky jacobsky merged commit 6c38d68 into godot-rust:master Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants