Skip to content

Conversation

@jdm
Copy link

@jdm jdm commented Jan 13, 2020

No description provided.

@jdm
Copy link
Author

jdm commented Jan 13, 2020

As far as I can tell its types are not exposed through the public API, so this shouldn't require a major version change.

@svartalf
Copy link
Owner

Hm, can you rebase from master branch? I forgot to add the CI execution on the PRs for this repo, at least we can do the tests.

Yet, I think I'll not be able to merge it right now, as there is probably the same issue as with #38 (comment); it will bump MSRV and it is probably undesired right now.
Do you have any specific needs to bump the mach version?

@jdm
Copy link
Author

jdm commented Jan 13, 2020

Only to avoid duplicate dependencies in Servo.

@svartalf
Copy link
Owner

@jdm wait, you mean Servo is using/going to use battery? Seems to be a noble reason to publish the new version then :D

@jdm
Copy link
Author

jdm commented Jan 13, 2020

It's used indirectly through wgpu-core, and we're apparently pulling battery in as part of servo/servo#25507. Duplicating mach isn't a big deal; make the choices that make sense for your primary consumers :)

@svartalf
Copy link
Owner

Ah, yes, barely found it: https://github.com/gfx-rs/wgpu/blob/9777bbba361690740c62fd294b968ac6f2340506/wgpu-core/Cargo.toml#L50-L51
For some reasons it is not shown at the "Reverse dependencies" page for the battery crate.

Bumping mach version is a long due task, in fact, I think we can manage it happen; probably by including some additional issues into the next release.
Let's postpone this PR and I'll check what I can do with it.

davidkna pushed a commit to davidkna/rust-battery that referenced this pull request Mar 24, 2024
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.0.0 to 4.1.0.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v4.0.0...v4.1.0)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.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.

2 participants