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.

Sign up for GitHub

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

Jump to bottom

Push integration docs only when releasing a new version #479

Closed
silvanocerza opened this issue Feb 23, 2024 · 0 comments · Fixed by #605
Closed

Push integration docs only when releasing a new version #479

silvanocerza opened this issue Feb 23, 2024 · 0 comments · Fixed by #605
Assignees
@anakin87
Labels
P1 topic:CI type:documentation Improvements or additions to documentation
Milestone
Integrations 2.1.0

Comments

@silvanocerza
Copy link
Contributor

As of now the docs for any integration are updated whenever a change is merged in main.

This is wrong as we might publish documentation for features or changes that have yet to be released.
We should change the workflow to push documentation only when there is a new release for that integration.
@silvanocerza silvanocerza added type:documentation Improvements or additions to documentation topic:CI labels Feb 23, 2024
@silvanocerza silvanocerza changed the title Push integration docs only releasing a new version Push integration docs only when releasing a new version Feb 23, 2024
@shadeMe shadeMe added this to the Integrations 2.1.0 milestone Feb 23, 2024
@masci masci added the P1 label Mar 15, 2024
@anakin87 anakin87 mentioned this issue Mar 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anakin87 anakin87

Labels
P1 topic:CI type:documentation Improvements or additions to documentation
Projects
None yet
Milestone
Integrations 2.1.0
Development

Successfully merging a pull request may close this issue.

CI: push docs when releasing a new version deepset-ai/haystack-core-integrations
4 participants
@masci @shadeMe @silvanocerza @anakin87