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

Update notify to 5.0.0-pre.13 from pre.11 #2993

Closed
wants to merge 1 commit into from

Conversation

deontologician
Copy link
Contributor

Objective

notify dep of bevy_assets is out of date

Solution

Bump the dependency to the latest version

@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Oct 20, 2021
@deontologician
Copy link
Contributor Author

Hmmm... this advisory check failure seems to be unrelated. I am guessing this advisory fails on main as well

@mockersf mockersf added A-Assets Load files from disk to use for things like images, models, and sounds C-Dependencies A change to the crates that Bevy depends on and removed S-Needs-Triage This issue needs to be labelled labels Oct 20, 2021
@mockersf
Copy link
Member

yup the advisories are related to other dependencies, not related here

bors bot pushed a commit that referenced this pull request Dec 9, 2021
# Objective

- there are a few new versions for `ron`, `winit`, `ndk`, `raw-window-handle`
- `cargo-deny` is failing due to new security issues / duplicated dependencies

## Solution

- Update our dependencies
- Note all new security issues, with which of Bevy direct dependency it comes from
- Update duplicate crate list, with which of Bevy direct dependency it comes from

`notify` is not updated here as it's in #2993
@@ -28,7 +28,7 @@ crossbeam-channel = "0.5.0"
anyhow = "1.0.4"
thiserror = "1.0"
downcast-rs = "1.2.0"
notify = { version = "=5.0.0-pre.11", optional = true }
notify = { version = "=5.0.0-pre.13", optional = true }
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
notify = { version = "=5.0.0-pre.13", optional = true }
notify = { version = "=5.0.0-pre.15", optional = true }

Copy link
Member

@alice-i-cecile alice-i-cecile left a comment

Choose a reason for hiding this comment

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

@deontologician bump the version number further (this went stale) and I'll be happy to merge it once you've tested to see that we didn't break hot-reloading.

See: https://crates.io/crates/notify/versions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Assets Load files from disk to use for things like images, models, and sounds C-Dependencies A change to the crates that Bevy depends on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants