Skip to content

[N/A] Make response autocompletion less intrusive - #175

Merged
mhidalgo-rai merged 2 commits into
mainfrom
mhidalgo-bdai/non-intrusive-autocomplete
Aug 19, 2025
Merged

[N/A] Make response autocompletion less intrusive#175
mhidalgo-rai merged 2 commits into
mainfrom
mhidalgo-bdai/non-intrusive-autocomplete

Conversation

@mhidalgo-rai

@mhidalgo-rai mhidalgo-rai commented Aug 18, 2025

Copy link
Copy Markdown
Collaborator

Precisely what the title says. Only override response headers during testing if not set by the user already.

Signed-off-by: Michel Hidalgo <mhidalgo@theaiinstitute.com>

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@mhidalgo-rai mhidalgo-rai self-assigned this Aug 18, 2025
@mhidalgo-rai mhidalgo-rai changed the title Make response autocompletion less intrusive [N/A] Make response autocompletion less intrusive Aug 18, 2025
@mhidalgo-rai
mhidalgo-rai marked this pull request as ready for review August 18, 2025 12:02
@coveralls

coveralls commented Aug 18, 2025

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 17073791994

Details

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 44.859%

Totals Coverage Status
Change from base Build 16969339390: 0.02%
Covered Lines: 2186
Relevant Lines: 4873

💛 - Coveralls

@khughes-bdai khughes-bdai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mhidalgo-rai
mhidalgo-rai merged commit 0f2a6a8 into main Aug 19, 2025
6 of 8 checks passed
@mhidalgo-rai
mhidalgo-rai deleted the mhidalgo-bdai/non-intrusive-autocomplete branch August 19, 2025 15:35
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.

4 participants