Skip to content
This repository was archived by the owner on Dec 31, 2025. It is now read-only.

Suppress unused method warning when using custom-godot feature#1084

Closed
mivort wants to merge 1 commit intogodot-rust:masterfrom
mivort:master
Closed

Suppress unused method warning when using custom-godot feature#1084
mivort wants to merge 1 commit intogodot-rust:masterfrom
mivort:master

Conversation

@mivort
Copy link
Copy Markdown
Contributor

@mivort mivort commented Sep 22, 2024

When custom-godot feature is used during build, the following warning gets produced:

warning: function `godot_version` is never used
   |
50 | fn godot_version() -> Option<semver::Version> {
   |    ^^^^^^^^^^^^^
   |

This PR adds #[allow(unused)] directive to suppress this warning, since it becomes unused when custom-godot feature is enabled.

When `custom-godot` feature is used during build, the following warning
gets produced:
```
warning: function `godot_version` is never used
   |
50 | fn godot_version() -> Option<semver::Version> {
   |    ^^^^^^^^^^^^^
   |
```

This PR adds `#[allow(unused)]` directive to suppress this warning,
since it becomes unused when `custom-godot` feature is enabled.
@GodotRust
Copy link
Copy Markdown

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

@Bromeon
Copy link
Copy Markdown
Member

Bromeon commented Sep 22, 2024

Thanks for the contribution!

Please note about this; CI would need some work before we can merge again.

@Bromeon Bromeon added the quality-of-life No new functionality, but improves ergonomics/internals label Sep 22, 2024
@Bromeon Bromeon added the status: buried Was open before archiving the repo, and will likely never be solved. label Dec 31, 2025
@Bromeon Bromeon closed this Dec 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

quality-of-life No new functionality, but improves ergonomics/internals status: buried Was open before archiving the repo, and will likely never be solved.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants