Skip to content

Implement 3 part MSRV policy#8999

Merged
cwfitzgerald merged 1 commit intogfx-rs:trunkfrom
cwfitzgerald:cw/msrv
Feb 8, 2026
Merged

Implement 3 part MSRV policy#8999
cwfitzgerald merged 1 commit intogfx-rs:trunkfrom
cwfitzgerald:cw/msrv

Conversation

@cwfitzgerald
Copy link
Copy Markdown
Member

Connections

Closes #8971

Description

This implements the plan laid out in #8971 (comment) and implements enforcement for it

Testing
CI.

@cwfitzgerald cwfitzgerald force-pushed the cw/msrv branch 3 times, most recently from 202b4d9 to dee7633 Compare February 4, 2026 22:35
Copy link
Copy Markdown
Collaborator

@inner-daemons inner-daemons left a comment

Choose a reason for hiding this comment

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

I have some unimportant questions

Due to complex dependants, we have three MSRV policies:

- `naga`, `wgpu-core`, `wgpu-hal`, and `wgpu-types`'s MSRV is **1.90**.
- `wgpu`'s MSRV is **1.87**
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

wgpu is a light crate, I don't think it would ever need a higher MSRV than core?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

There is still quite a bit of machinery in it, so we might want to bump it for various rust utilities.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I would opine that this wouldn't justify the complexity but I think that discussion has already been had.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's not really that much complexity, just an extra set of CI checks, essentially.

Copy link
Copy Markdown
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

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

awesome, thank you so much!

@cwfitzgerald cwfitzgerald merged commit 04746e7 into gfx-rs:trunk Feb 8, 2026
58 checks passed
@cwfitzgerald cwfitzgerald deleted the cw/msrv branch February 8, 2026 22:04
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.

Consider Max MSRV Policy

3 participants