Skip to content

Update to Godot 3.3 #728

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 1 commit into from
Apr 28, 2021
Merged

Update to Godot 3.3 #728

merged 1 commit into from
Apr 28, 2021

Conversation

jdfreder
Copy link
Contributor

Hi, thanks for the great bindings!

Now that Godot 3.3 is the stable branch I thought that it made sense to update my local copy. Just upstreaming it incase it saves you time.

Tested:
cargo test
I also verified that the hello-world example runs in the Godot IDE

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Could you keep the scope to updating api.json and revert the changes to the version numbers everywhere? We only bump them when we actually make releases. Having them randomly bumped in individual PRs would be messy to keep track of, and troublesome should we ever want to revert one of them.

You'll also want to update the API documentation XMLs from Godot's repo, in addition to api.json, so the generated docs would be up-to-date. Keep in mind that their master branch is for 4.0. You'll want to checkout the release commit and copy from there.

@Bromeon
Copy link
Member

Bromeon commented Apr 27, 2021

If I understood it right, we would still be compatible with Godot 3.2., as 3.3 is technically a 3.2. release, with a large enough scope to justify minor version bump (but no breaking changes).

If that's the case, maybe we should mention this in the ReadMe.

@ghost
Copy link

ghost commented Apr 27, 2021

If I understood it right, we would still be compatible with Godot 3.2., as 3.3 is technically a 3.2. release, with a large enough scope to justify minor version bump (but no breaking changes).

If that's the case, maybe we should mention this in the ReadMe.

Yes, so it isn't really necessary to change the ReadMe: our minimum supported stable branch (with API.json replacement) is still 3.2. Changing the default API version does not change that. In any case, the crate version on master shouldn't really be bumped in this. We might want to consider releasing a further patch release in 0.9.x, though, since 0.10 might still take a while.

@jdfreder jdfreder force-pushed the master branch 2 times, most recently from 50422ae to 412efbe Compare April 27, 2021 14:54
@jdfreder
Copy link
Contributor Author

Thanks for the PR! Could you keep the scope to updating api.json and revert the changes to the version numbers everywhere? We only bump them when we actually make releases. Having them randomly bumped in individual PRs would be messy to keep track of, and troublesome should we ever want to revert one of them.

You'll also want to update the API documentation XMLs from Godot's repo, in addition to api.json, so the generated docs would be up-to-date. Keep in mind that their master branch is for 4.0. You'll want to checkout the release commit and copy from there.

Done

@jdfreder jdfreder requested a review from a user April 28, 2021 03:53
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks!

bors r+

@bors
Copy link
Contributor

bors bot commented Apr 28, 2021

Build succeeded:

@bors bors bot merged commit 6a1af86 into godot-rust:master Apr 28, 2021
@ghost ghost mentioned this pull request May 1, 2021
7 tasks
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