Skip to content

Remove deprecated symbols from before v0.1 #808

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 3 commits into from
Jul 22, 2024
Merged

Conversation

Bromeon
Copy link
Member

@Bromeon Bromeon commented Jul 22, 2024

These symbols were already deprecated before v0.1 and have been kept in the initial crates.io release for compatibility. However, some of them caused confusion for IDE imports (same symbol in 2 modules) as well as for Rustdoc intra-doc links.

The export flag radians (replaced with radians_as_degrees) stays because it's deprecated in Godot, and when retaining compatibility with 4.1, we need to support it. However, it keeps creating a deprecation warning for >= 4.2.

The old GString::chars_[un]checked methods stay for now, but will be removed alongside 4.0 support.

@Bromeon Bromeon added quality-of-life No new functionality, but improves ergonomics/internals breaking-change Requires SemVer bump labels Jul 22, 2024
@Bromeon Bromeon force-pushed the qol/remove-deprecated branch from 391711c to c076f81 Compare July 22, 2024 20:21
@Bromeon Bromeon added this to the 0.2 milestone Jul 22, 2024
@Bromeon Bromeon enabled auto-merge July 22, 2024 20:23
@Bromeon Bromeon added this pull request to the merge queue Jul 22, 2024
github-merge-queue bot pushed a commit that referenced this pull request Jul 22, 2024
Remove deprecated symbols from before v0.1
@GodotRust
Copy link

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdext/pr-808

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 22, 2024
@Bromeon Bromeon enabled auto-merge July 22, 2024 20:44
@Bromeon Bromeon added this pull request to the merge queue Jul 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 22, 2024
@Bromeon Bromeon force-pushed the qol/remove-deprecated branch 2 times, most recently from 0511454 to d75810c Compare July 22, 2024 21:17
@Bromeon Bromeon added this pull request to the merge queue Jul 22, 2024
Merged via the queue into master with commit 421324a Jul 22, 2024
15 checks passed
@Bromeon Bromeon deleted the qol/remove-deprecated branch July 22, 2024 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change Requires SemVer bump quality-of-life No new functionality, but improves ergonomics/internals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants