Skip to content
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

CI: Only MSRV-test published ash and ash-window crates #677

Merged
merged 1 commit into from
Nov 7, 2022

Conversation

MarijnS95
Copy link
Collaborator

bytemuck recently bumped its MSRV to 1.60 which is incompatible with what we're currently advertising. Fortunately this doesn't affect MSRV of the publicly published crates, but only dev-dependencies inside ash-window via winit: make sure the CI doesn't reject this.

$ cargo tree -i bytemuck
bytemuck v1.12.3
├── image v0.24.4
│   └── examples v0.1.0 (ash/examples)
├── safe_arch v0.5.2
│   └── tiny-skia v0.7.0
│       └── sctk-adwaita v0.4.3
│           └── winit v0.27.5
│               └── examples v0.1.0 (ash/examples)
│               [dev-dependencies]
│               └── ash-window v0.11.0 (ash/ash-window)
│                   └── examples v0.1.0 (ash/examples)
├── tiny-skia v0.7.0 (*)
└── tiny-skia-path v0.7.0
    └── tiny-skia v0.7.0 (*)

`bytemuck` recently bumped its MSRV to 1.60 which is incompatible with
what we're currently advertising.  Fortunately this doesn't affect MSRV
of the publicly published crates, but only `dev-dependencies` inside
`ash-window` via `winit`: make sure the CI doesn't reject this.
@Ralith Ralith merged commit 373aee5 into master Nov 7, 2022
@Ralith Ralith deleted the ci-msrv-only-published branch November 7, 2022 19:58
MarijnS95 added a commit that referenced this pull request Nov 18, 2022
`bytemuck` recently bumped its MSRV to 1.60 which is incompatible with
what we're currently advertising.  Fortunately this doesn't affect MSRV
of the publicly published crates, but only `dev-dependencies` inside
`ash-window` via `winit`: make sure the CI doesn't reject this.
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