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

upgrade to bevy 0.12 #221

Merged
merged 11 commits into from
Nov 5, 2023
Merged

upgrade to bevy 0.12 #221

merged 11 commits into from
Nov 5, 2023

Conversation

raffaeleragni
Copy link
Contributor

to update cargo when bevy 0.12 is out

@paul-hansen paul-hansen mentioned this pull request Nov 4, 2023
@frewsxcv
Copy link
Contributor

frewsxcv commented Nov 4, 2023

Bevy 0.12 was just released!

@raffaeleragni raffaeleragni marked this pull request as ready for review November 4, 2023 21:29
@raffaeleragni
Copy link
Contributor Author

yea, cargo updated

src/lib.rs Outdated
@@ -144,7 +144,7 @@ fn compare_descriptors(a: &ImageSampler, b: &ImageSampler) -> bool {
match (a, b) {
(ImageSampler::Default, ImageSampler::Default) => true,
(ImageSampler::Descriptor(descriptor_a), ImageSampler::Descriptor(descriptor_b)) => {
descriptor_a == descriptor_b
descriptor_a.label == descriptor_b.label

This comment was marked as resolved.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

compared all of them

src/lib.rs Outdated Show resolved Hide resolved
raffaeleragni and others added 2 commits November 5, 2023 13:11
Co-authored-by: Matthew S. Klosak <9565094+msklosak@users.noreply.github.com>
src/lib.rs Outdated Show resolved Hide resolved
src/systems.rs Show resolved Hide resolved
@mvlabat
Copy link
Owner

mvlabat commented Nov 5, 2023

@raffaeleragni thanks a lot for the PR! There are a few CI warnings, but I'll take it from here. (I'll push directly to your branch, I hope you won't mind.)

@raffaeleragni
Copy link
Contributor Author

no prob

@mvlabat mvlabat merged commit de7497a into mvlabat:main Nov 5, 2023
19 checks passed
@raffaeleragni raffaeleragni deleted the bevy_0_12 branch November 5, 2023 23:01
Schmarni-Dev pushed a commit to Schmarni-Dev/bevy_egui that referenced this pull request Dec 23, 2023
* upgrade to bevy 0.12

* upgrade

* bevy release

* Update src/lib.rs

Co-authored-by: Matthew S. Klosak <9565094+msklosak@users.noreply.github.com>

* compare all fields

* use real time

* Simplify the compare_descriptors implementation

* Fix compiling in WASM

* Fix formatting

* Fix web-sys version just in the CI instead

* Remove redundant dev-dependencies features

---------

Co-authored-by: Matthew S. Klosak <9565094+msklosak@users.noreply.github.com>
Co-authored-by: mvlabat <mvlabat@gmail.com>
Schmarni-Dev pushed a commit to Schmarni-Dev/bevy_egui that referenced this pull request Dec 23, 2023
* upgrade to bevy 0.12

* upgrade

* bevy release

* Update src/lib.rs

Co-authored-by: Matthew S. Klosak <9565094+msklosak@users.noreply.github.com>

* compare all fields

* use real time

* Simplify the compare_descriptors implementation

* Fix compiling in WASM

* Fix formatting

* Fix web-sys version just in the CI instead

* Remove redundant dev-dependencies features

---------

Co-authored-by: Matthew S. Klosak <9565094+msklosak@users.noreply.github.com>
Co-authored-by: mvlabat <mvlabat@gmail.com>
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.

7 participants