Skip to content

Refactor tests to be feature flag bit-width agnostic#5155

Merged
cwfitzgerald merged 1 commit intogfx-rs:trunkfrom
atlv24:features-u128
Jan 28, 2024
Merged

Refactor tests to be feature flag bit-width agnostic#5155
cwfitzgerald merged 1 commit intogfx-rs:trunkfrom
atlv24:features-u128

Conversation

@atlv24
Copy link
Copy Markdown
Collaborator

@atlv24 atlv24 commented Jan 28, 2024

Connections
Part of an incoming series of PRs needed by #5123
Others in this series: #5154
Ultimately for bevy meshlets pipeline bevyengine/bevy#10164

Description
Refactor tests to be feature flag bit-width agnostic. This is to make it possible to easily change Features from u64 to u128 in the future. This will be needed because we are out of feature flag space to add new ones.

Testing
This only affects existing tests, and the tests still pass.

Checklist

  • Run cargo fmt.
  • 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.

@atlv24 atlv24 requested a review from a team as a code owner January 28, 2024 01:02
@atlv24 atlv24 mentioned this pull request Jan 28, 2024
6 tasks
Copy link
Copy Markdown
Member

@cwfitzgerald cwfitzgerald left a comment

Choose a reason for hiding this comment

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

Nice, we've needed this for a while!

@cwfitzgerald cwfitzgerald enabled auto-merge (squash) January 28, 2024 01:17
@cwfitzgerald cwfitzgerald merged commit 0003db1 into gfx-rs:trunk Jan 28, 2024
@atlv24 atlv24 mentioned this pull request Mar 13, 2024
6 tasks
@atlv24 atlv24 mentioned this pull request Apr 15, 2024
6 tasks
@atlv24 atlv24 deleted the features-u128 branch June 29, 2025 21:45
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