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

Handle __MACOSX #11

Merged
merged 1 commit into from
Jun 22, 2024
Merged

Handle __MACOSX #11

merged 1 commit into from
Jun 22, 2024

Conversation

KoduIsGreat
Copy link
Contributor

@KoduIsGreat KoduIsGreat commented Jun 19, 2024

Hi, thanks for the library

Theres an issue with zips compressed on macos.
Essentially zips on Macs include a __MACOSX dir with pointers to all the files, this causes the numerical checks for number of file types by extension to always be >1.

I've fixed the issue

go.mod Outdated Show resolved Hide resolved
scanner.go Outdated Show resolved Hide resolved
@KoduIsGreat
Copy link
Contributor Author

@twpayne Addressed comments, let me know if you need any further changes

@KoduIsGreat KoduIsGreat requested a review from twpayne June 20, 2024 02:31
util_test.go Show resolved Hide resolved
util_test.go Show resolved Hide resolved
@KoduIsGreat KoduIsGreat requested a review from twpayne June 21, 2024 20:52
Copy link
Owner

@twpayne twpayne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making the changes! Looks good. Please address the remaining comment and squash all the commits into one. Then this will be good to merge.

.gitignore Outdated Show resolved Hide resolved
@KoduIsGreat
Copy link
Contributor Author

@twpayne complete! thanks :)

@twpayne twpayne merged commit e066f98 into twpayne:master Jun 22, 2024
3 checks passed
@twpayne
Copy link
Owner

twpayne commented Jun 22, 2024

Thanks! I've tagged v0.0.2 which includes your changes.

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.

None yet

2 participants