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 cargo-check-external-types rustc version #41

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

cpu
Copy link
Contributor

@cpu cpu commented May 29, 2024

👋 As promised, when cargo-check-external-types makes an update I'm opening PRs to fix the builds :-)

Note: there seems to be another unrelated breakage with nightly clippy, a finding from unexpected-cfgs. I'm not sure what the right fix is in this case: it seems like "exact_size_is_empty" is being used as a feature gate for the unstable nightly feature (rust-lang/rust#35428) and I'm not sure how to square that requirement with the new lint. Suggestions welcome!

The upstream project cut a 0.1.12 release that now pins Rust
nightly-2024-05-01. This commit updates CI to match.
chifflier added a commit that referenced this pull request Jun 14, 2024
It seems we have no easy way to detect the use of the unstable feature
"exact_size_is_empty", and this function is not required since it defaults
to checking if `.len() == 0`.
Additionally, this function is not used, so removing it.
@chifflier
Copy link
Member

Hi,
I checked but it seems that there is no easy way for us to check for the nightly feature. Anyway, the function was mostly provided as a helper, we do not use it.
Removed in 382dacf

@chifflier chifflier merged commit 64502e4 into rusticata:master Jun 14, 2024
10 of 13 checks passed
@chifflier
Copy link
Member

Applied, thanks!

@cpu cpu deleted the cpu-fix-build branch June 14, 2024 13:08
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