Skip to content

Remove legacy sidecar CLI and HTTP methods#442

Merged
gjreda merged 2 commits into
mainfrom
411-remove-sidecar-cli-for-ingest
Aug 30, 2023
Merged

Remove legacy sidecar CLI and HTTP methods#442
gjreda merged 2 commits into
mainfrom
411-remove-sidecar-cli-for-ingest

Conversation

@gjreda

@gjreda gjreda commented Aug 29, 2023

Copy link
Copy Markdown
Collaborator

fixes #411 and fixes #427

@codecov

codecov Bot commented Aug 29, 2023

Copy link
Copy Markdown

Codecov Report

Merging #442 (8af7f87) into main (a322cd0) will increase coverage by 0.31%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #442      +/-   ##
==========================================
+ Coverage   77.78%   78.09%   +0.31%     
==========================================
  Files         188      188              
  Lines       11527    11481      -46     
  Branches     1116     1116              
==========================================
  Hits         8966     8966              
+ Misses       2546     2500      -46     
  Partials       15       15              
Files Changed Coverage Δ
python/main.py 0.00% <0.00%> (ø)
python/sidecar/http.py 82.82% <ø> (+2.82%) ⬆️
python/web.py 0.00% <ø> (ø)
src/api/api-types.ts 0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Comment thread src/api/api-types.ts
@@ -20,46 +20,55 @@ export type Detail = ValidationError[];
* An enumeration.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I'm not sure why codegen changed this file, but from what I can tell the changes are just the order in which things are defined in the file.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I've seen this before. It's a bit anoying.

@gjreda gjreda marked this pull request as ready for review August 29, 2023 19:00
@gjreda gjreda requested review from cguedes and danvk August 29, 2023 19:00

@cguedes cguedes left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🌮

Comment thread src/api/api-types.ts
@@ -20,46 +20,55 @@ export type Detail = ValidationError[];
* An enumeration.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I've seen this before. It's a bit anoying.

@gjreda gjreda merged commit 7d005cd into main Aug 30, 2023
@gjreda gjreda deleted the 411-remove-sidecar-cli-for-ingest branch August 30, 2023 21:37
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.

Remove backend API methods for AI functions Remove sidecar CLI for Ingest

2 participants