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

Add flag to skip renaming files #175

Merged
merged 3 commits into from
Jan 3, 2024
Merged

Add flag to skip renaming files #175

merged 3 commits into from
Jan 3, 2024

Conversation

naglepuff
Copy link
Collaborator

@naglepuff naglepuff commented Dec 13, 2023

Fix #171

Add an option to the run command to keep original file names, in the case where users are confident that their image file names do not contain PHI but would be helpful to preserve after redaction.

To use:

imagedephi run path/to/images -o output/path --skip-rename

@naglepuff naglepuff requested a review from manthey December 13, 2023 17:00
@manthey
Copy link
Contributor

manthey commented Jan 3, 2024

Because of an upstream package change, you'll need #177 to allow CI to pass again.

@naglepuff
Copy link
Collaborator Author

@manthey I merged branch harden-some-tests (the source for #177) into this branch to fix CI. Not sure if you wanted to merge #177 before this PR.

@manthey
Copy link
Contributor

manthey commented Jan 3, 2024

@manthey I merged branch harden-some-tests (the source for #177) into this branch to fix CI. Not sure if you wanted to merge #177 before this PR.

I'll have you approve #177 and then I'll merge it.

@naglepuff naglepuff merged commit 211da76 into main Jan 3, 2024
4 checks passed
@naglepuff naglepuff deleted the 171-skip-rename branch January 3, 2024 20:23
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.

Specifying target file name
2 participants