NotificationsYou must be signed in to change notification settings
Fork989
Star12k

Add password#3876

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Draft

Coniferish wants to merge8 commits intomain

base:main

Choose a base branch

fromfix_password

Draft

Add password#3876

Coniferish wants to merge8 commits intomainfromfix_password

Conversation

Copy link

Contributor

Coniferish commentedJan 18, 2025

No description provided.

ppradosand others added8 commits

January 17, 2025 16:03

Add password with PDF files

d2cb518

Add TU

10d1f39

Merge with the last version

bc19db2

linting

e2d82e5

fix type and default

5fc2f7e

fix: linting and small bugs

c252fc1

make tidy

623081b

ignore check error

1173c0b

Coniferish changed the title~~Fix password~~Add password

Jan 19, 2025

Copy link

ContributorAuthor

Coniferish commentedJan 19, 2025

@pprados, I went ahead and rebased your branch and made some linting fixes for your branch. It looks like the tests that are failing right now stem from not having your PR inunstructured_inference not merged yet. So it seems like we just need to get your changes in that repo and then we can get this merged. For future reference, though, you can can run most of the tests that our CI runs withmake test andmake tidy.