v9.0.0: New version needs

[Its-Just-Nans]

New major needs (breaking changes)

See comment below

New minor needs

• add #[must_use] on - possible to do on MINOR version feat: add must_use #727
  • method zip::write::FileOptions::into_full_options in /home/runner/work/zip2/zip2/src/write.rs:722
  • method zip::write::ZipWriter::set_auto_large_file in /home/runner/work/zip2/zip2/src/write.rs:1023
  • method zip::System::from_version_made_by in /home/runner/work/zip2/zip2/src/types.rs:91

Others changes

• refactor: improve part of the code with clippy help #725

[Its-Just-Nans]

@Pr0methean

If you have the time, could you review and approve

• doc: improve crate documentation #708
• feat: add crc32 ignore option #710
• feat: path related code in single file #712
• feat: improve and fix extended timestamp extra field parsing #713
• refactor: reorganize code #714

Maybe they will be merge conflicts between them but I can fix that if you approve them

thanks

[Its-Just-Nans]

To fix #740

• fix: allow for [u8] as filename #775
• then make a breaking change for file_name to be lazy computed with a Result<>
  • from what I see, the file_name is not used that much internally, it's mostly used for the public API

[Its-Just-Nans]

To fix #738

• we are stuck with pub(crate) fn initialize_local_block which requires a String because ZipFileData requires a string

-> should do a breaking change

[Its-Just-Nans]

ZipFileData is only used on the API with the trait HasZipMetadata. But it seems that this trait WAS used, but not anymore since we mostly use ZipFile

• Add by_name_seek() for Stored zips zip-old#369
• feat: Add by_name_seek() for Stored zips  #69

EDIT

ZipFileData can be used externally with . but we could remove HasZipMetadata https://github.com/zip-rs/zip2/compare/remove-haszipmetada

[Its-Just-Nans]

TODO for v9

• merge refactor!: store the file_name_raw only as key #789 (remove file_name_raw in ZipFileData, keep only file_name)
• merge refactor: don't store converted filenames; convert on demand instead; and don't use Arc since it only deduplicates when copying to same path in another archive #807 (remove file_name in ZipFileData)
• rewrite ExtraField
  • rewrite ExtraField logic
  • add #[non_exhaustive] on the pub enum ExtraField - refactor: add non exhaustive to extra fields #793
• do the tiny task (see first comment)
  • make ZipData/ZipFile #[non_exhaustive] refactor: Remove useless compressed level #791
  • remove the #[macro_export] refactor: remove macro export #792
• introduce Draft: iterable zip #487

[Pr0methean]

We should also delete deprecated methods, fields and enum variants:

• ZipFile::sanitized_name
• ArchiveOffset::FromCentralDirectory
• impl TryFrom for time::OffsetDateTime and for time::PrimitiveDateTime
• ZipArchive::zip64_comment

[Pr0methean]

#802 will remove the above.