Ensure no warnings on various targets#7245
Ensure no warnings on various targets#7245ShaharNaveh wants to merge 12 commits intoRustPython:mainfrom
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Code has been automatically formatted The code in this PR has been formatted using:
git pull origin disallow-warnings |
| #[pyarg(positional, default)] | ||
| #[pystruct_sequence(unnamed)] | ||
| pub st_mtime_int: libc::time_t, | ||
| #[cfg_attr(target_env = "musl", allow(deprecated))] |
There was a problem hiding this comment.
No matter what I do, the deprecation warning still shows. @youknowone I believe we have a bug related to #[cfg_attr(...)] with #[pystruct_sequence(...)] (and possibly related to when used with #[pyarg(...)])
ATM build on android yield an "unused import" warning. This PR aims to make sure warnings like this don't appear from time to time