app-dirs-rs / app_dirs2

A maintained fork of the app-dirs-rs project.
MIT License
35 stars 7 forks source link

android: Upgrade to `jni 0.21` #35

Closed MarijnS95 closed 1 year ago

MarijnS95 commented 1 year ago

Cc @rib

rib commented 1 year ago

yeah, those changes look good to me.

the clippy issues look completely unrelated to this, so seems fine to ignore those here.

MarijnS95 commented 1 year ago

The clippy issues look completely unrelated to this, so seems fine to ignore those here.

New Rust version, stricter (really: more complete/correct/accurate) clippy lints, for better or for worse :)

kornelski commented 1 year ago

Thank you

MarijnS95 commented 1 year ago

Thank you @kornelski for the quick release!

Fwiw I see additional clippy changes, are you running a configuration locally that differs from the CI?

kornelski commented 1 year ago

I just run clippy manually and selectively apply what seems sensible

MarijnS95 commented 1 year ago

Would be great to list those in the CI and/or lib.rs so that contributors can proactively address them too.

kornelski commented 1 year ago

I'd rather not add stronger clippy requirements in the CI, because these are suggestions, not absolute requirements. If the clippy build in CI is confusing or getting in the way of PRs, I'd rather remove it from CI.

MarijnS95 commented 1 year ago

Just asking how/where you'd manage to find more: manually requesting them on the command line or running a newer-than-stable toolchain?

kornelski commented 1 year ago

I've added -W clippy::pedantic to the flags.