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

added the file extension to the ExtractAction model, since this data i used by renamer and it is for most files also data that we ahve in the fileformats.yml #17

Merged
merged 2 commits into from
Nov 15, 2023

Conversation

Magniler
Copy link
Contributor

This is some functionality that Unarchiver requires.

Some of the tools we use to unpack certain fileformats are very strict as to what file extensions the files that they work with can have, so we need to update the filles which have the wrong extensions

…i used by renamer and it is for most files also data that we ahve in the fileformats.yml
@Magniler Magniler force-pushed the add-suffix-to-extract-action branch from 09ea065 to 2b075c2 Compare November 15, 2023 09:10
@MatteoCampinoti94
Copy link
Member

I'd say to change the attribute to just "extension". It's already deep inside the File object (File.action_data.extract.extension), no need tobe extra verbose :)

@Magniler Magniler merged commit d83af84 into main Nov 15, 2023
3 checks passed
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.

2 participants