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

move package under haystack_integrations/* #212

Merged
merged 2 commits into from
Jan 17, 2024
Merged

Conversation

masci
Copy link
Contributor

@masci masci commented Jan 15, 2024

Part of #208

See #100 for details

Note: this PR requires deepset-ai/haystack#6714 that's not present in beta4, need to wait for beta5 to let the tests pass

@masci masci requested a review from a team as a code owner January 15, 2024 15:39
@masci masci requested review from vblagoje and removed request for a team January 15, 2024 15:39
@vblagoje
Copy link
Member

@masci Do we have to do this for all integrations, and this is a prototype of how to do it for other integrations?

@masci
Copy link
Contributor Author

masci commented Jan 15, 2024

@vblagoje we have to do similar changes to all the integrations, see #100 it contains the list

@masci masci merged commit 09fb70d into main Jan 17, 2024
7 checks passed
@masci masci deleted the massi/opensearch-namespace branch January 17, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants