Skip to content

Use default-features, not default_features#3746

Merged
madsmtm merged 1 commit into
rust-windowing:masterfrom
waywardmonkeys:use-default-dash-features-not-underscore
Jun 22, 2024
Merged

Use default-features, not default_features#3746
madsmtm merged 1 commit into
rust-windowing:masterfrom
waywardmonkeys:use-default-dash-features-not-underscore

Conversation

@waywardmonkeys
Copy link
Copy Markdown
Contributor

The latter syntax is deprecated and will be removed in Rust 2024 edition. This also generates a warning with current versions of Rust.

  • Tested on all platforms changed
  • Added an entry to the changelog module if knowledge of this change could be valuable to users
  • Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • Created or updated an example program if it would help users understand this functionality
  • Updated feature matrix, if new features were added or implemented

The latter syntax is deprecated and will be removed in Rust
2024 edition. This also generates a warning with current
versions of Rust.
Copy link
Copy Markdown
Member

@madsmtm madsmtm left a comment

Choose a reason for hiding this comment

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

Thanks!

@madsmtm madsmtm merged commit c6c4395 into rust-windowing:master Jun 22, 2024
@waywardmonkeys waywardmonkeys deleted the use-default-dash-features-not-underscore branch June 22, 2024 10:03
kchibisov pushed a commit that referenced this pull request Jul 16, 2024
The latter syntax is deprecated and will be removed in Rust
2024 edition. This also generates a warning with current
versions of Rust.
kchibisov pushed a commit that referenced this pull request Jul 16, 2024
The latter syntax is deprecated and will be removed in Rust
2024 edition. This also generates a warning with current
versions of Rust.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants