Skip to content

Conversation

@D1m7asis
Copy link
Contributor

@D1m7asis D1m7asis commented Oct 20, 2025

Summary

Added aimlapi-haystack to the integrations table in the main README with links to its documentation, PyPI package, and CI workflow badge.

Related Issues

Proposed Changes

  • Updated README.md to include:
    • Integration path: integrations/aimlapi/
    • PyPI badge: aimlapi-haystack
    • CI badge: actions/workflows/aimlapi.yml

How did you test it?

  • Verified markdown rendering locally and in GitHub preview (table layout, links, and badges display correctly).

Notes for the reviewer

  • This is a documentation-only change; no code modifications or tests affected.

Checklist

  • I’ve read the Contributing Guidelines and Code of Conduct
  • Linked the related issue
  • Used a proper conventional commit (docs:)
  • Verified all links and badges placeholders work correctly

Included aimlapi-haystack in the integrations table with links to its documentation, PyPI package, and CI status badge.
@D1m7asis D1m7asis requested a review from a team as a code owner October 20, 2025 16:40
@D1m7asis D1m7asis requested review from julian-risch and removed request for a team October 20, 2025 16:40
@D1m7asis D1m7asis mentioned this pull request Oct 20, 2025
9 tasks
Copy link
Member

@julian-risch julian-risch left a comment

Choose a reason for hiding this comment

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

Hi @D1m7asis Thanks for opening the pull request! Looks good to me in general but we'll need https://github.com/deepset-ai/haystack-core-integrations/actions/workflows/aimlapi.yml to be created first. After that, this PR should be ready to be merged. 👍

@D1m7asis D1m7asis force-pushed the aimlapi-inventory-section branch 3 times, most recently from cf94fd7 to 33a761f Compare October 21, 2025 12:02
Introduces a new GitHub Actions workflow for AIMLAPI integration testing. Updates example scripts to add noqa comments for print statements and refines message list construction. Modifies AIMLAPIChatGenerator to include an explicit 'openai_endpoint' parameter and clarifies a type ignore comment.
@D1m7asis D1m7asis force-pushed the aimlapi-inventory-section branch from 10c75b0 to f06b3b5 Compare October 21, 2025 12:34
@github-actions github-actions bot added the type:documentation Improvements or additions to documentation label Oct 21, 2025
@D1m7asis D1m7asis requested a review from julian-risch October 21, 2025 12:37
@D1m7asis
Copy link
Contributor Author

@julian-risch Hi! Added the aimlapi.yml workflow, CI now runs successfully for the AIMLAPI integration. All examples and README references updated accordingly. Should be ready to merge.

@mpangrazzi
Copy link
Contributor

Hey @D1m7asis sorry, didn't see this. Already done in #2520.

@mpangrazzi mpangrazzi closed this Nov 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:aimlapi topic:CI type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AIMLAPI: missing tasks

3 participants