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 password with PDF files #392

Merged
merged 18 commits into from
Jan 30, 2025

Conversation

pprados
Copy link
Contributor

@pprados pprados commented Oct 15, 2024

Add password with PDF files.
Must be combined with PR 3721 in unstructured

@pprados pprados marked this pull request as ready for review October 15, 2024 12:20
@pprados
Copy link
Contributor Author

pprados commented Oct 24, 2024

@badGarnet can you activate the review process ?

Copy link
Contributor

@cragwolfe cragwolfe left a comment

Choose a reason for hiding this comment

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

while this is not doing much besides argument passing, please add a test showing that works.

@pprados
Copy link
Contributor Author

pprados commented Oct 29, 2024

@cragwolfe
I add a test

@pprados
Copy link
Contributor Author

pprados commented Oct 31, 2024

@cragwolfe
Can you revu this code ?

@pprados
Copy link
Contributor Author

pprados commented Nov 20, 2024

Hey, @cragwolfe, can you revu this code ? Or just launch the workflow?

@pprados
Copy link
Contributor Author

pprados commented Dec 10, 2024

@cragwolfe
I resynchronize the code with the main branch. Can you activate the workflow ?

@Coniferish Coniferish mentioned this pull request Jan 19, 2025
@cragwolfe
Copy link
Contributor

@pprados , please bump https://github.com/Unstructured-IO/unstructured-inference/blob/main/unstructured_inference/__version__.py and edit the CHANGELOG.md to match. thanks for the contribution!

@pprados
Copy link
Contributor Author

pprados commented Jan 30, 2025

I will adjust the associated PR in unstructured, once it has been validated.

@pprados
Copy link
Contributor Author

pprados commented Jan 30, 2025

@cragwolfe
I fix the lint. You can restart the workflow.

@pprados
Copy link
Contributor Author

pprados commented Jan 30, 2025

@cragwolfe
I can't do anything about this.

Run aws-actions/configure-aws-credentials@v4
  with:
    audience: sts.amazonaws.com
Error: Input required and not supplied: aws-region

@Coniferish Coniferish merged commit cd9ea8b into Unstructured-IO:main Jan 30, 2025
24 of 28 checks passed
@Coniferish
Copy link
Collaborator

Merged! Thanks @pprados for the work!
Next up is to get the unstructured packaged updated with these changes and your PR in that repo merged

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.

4 participants