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 #3876

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from
Draft

Add password #3876

wants to merge 8 commits into from

Conversation

Coniferish
Copy link
Collaborator

No description provided.

@Coniferish Coniferish changed the title Fix password Add password Jan 19, 2025
@Coniferish
Copy link
Collaborator Author

@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 in unstructured_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 with make test and make tidy.

@pprados
Copy link

pprados commented Jan 21, 2025

@Coniferish

The PR392 in unstructured-inference MUST be validated before this one.

can you take a look?

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