-
Notifications
You must be signed in to change notification settings - Fork 389
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
feat: preview & extract file formats supported by 7-zip #1926
base: main
Are you sure you want to change the base?
Conversation
7be32c8
to
2ef7d52
Compare
80b1a83
to
ce8013c
Compare
@sxyazi This PR has been completed :) Would you like to review it? |
ea6e50c
to
428c922
Compare
Thanks for the PR! I noticed you referenced the supported formats of The supported formats often vary between them. For default configuration, we need to consider the formats supported across all platforms, which means focusing on the smallest common subset - supported by both This PR includes many formats I've never seen before, which makes testing them much more difficult. I think we can limit it to common formats that have been verified to work. This would also simplify the rules, especially considering that |
a8e272a
to
cfa096f
Compare
@sxyazi I just removed Additionally, |
f96929b
to
0f9a319
Compare
References: