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

Remove stale references to d3d12 bindings wrapper crate #6675

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

MarijnS95
Copy link
Contributor

Connections

Description
While looking through some recent activity for the D3D12 crate, in particular a spelling fix for the crates.io link to d3d12 (#6116), I realized that I completely forgot to remove these references to the now unused and removed d3d12 crate in #5956.

Testing
N/A

Checklist

  • Run cargo fmt.
  • Run taplo format.
  • Run cargo clippy. If applicable, add:
    • --target wasm32-unknown-unknown
    • --target wasm32-unknown-emscripten
  • Run cargo xtask test to run tests.
  • Add change to CHANGELOG.md. See simple instructions inside file.

Further work

In addition the deprecation notice by #6157, merged to the v22 branch, seems to have never been published to crates.io, hence the proposed d3d12-v22.1.1 tag has never materialized leaving the unpublished repository URL invalid.
(Furthermore I think the notice could have pointed users to the windows crate directly, in addition to referring them there via #5956)

While looking through some recent activity for the D3D12 crate, in
particular a spelling fix for the crates.io link to `d3d12`, I realized
that I completely forgot to remove these references to the now unused
and removed `d3d12` crate in gfx-rs#5956.
@MarijnS95 MarijnS95 requested a review from a team as a code owner December 6, 2024 09:00
Copy link
Member

@teoxoy teoxoy left a comment

Choose a reason for hiding this comment

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

Thanks!

@teoxoy teoxoy changed the title REAMDE: Remove stale references to d3d12 bindings wrapper crate Remove stale references to d3d12 bindings wrapper crate Dec 6, 2024
@teoxoy teoxoy merged commit 76f307c into gfx-rs:trunk Dec 6, 2024
27 checks passed
@MarijnS95 MarijnS95 deleted the remove-d3d12 branch December 6, 2024 10:58
@MarijnS95
Copy link
Contributor Author

MarijnS95 commented Dec 6, 2024

CC @cwfitzgerald for publishing the deprecation note to https://crates.io/crates/d3d12, perhaps?

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