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

Merge from private compass-sdk #11

Merged
merged 1 commit into from
Jul 3, 2024
Merged

Conversation

benrules3
Copy link
Contributor

Changes are being added first to the internal SDK and then migrated here. This is a placeholder showing the main content of the incoming changes.

@benrules3 benrules3 force-pushed the benrules1/merge_sdk_fixes branch 2 times, most recently from 0d3c819 to 4783ce6 Compare July 3, 2024 02:58
@benrules3 benrules3 changed the title [WIP] merge sdk fixes Merge from private compass-sdk Jul 3, 2024
@benrules3 benrules3 marked this pull request as ready for review July 3, 2024 16:06
@benrules3 benrules3 requested a review from a team as a code owner July 3, 2024 16:06
@benrules3 benrules3 force-pushed the benrules1/merge_sdk_fixes branch 2 times, most recently from 910db02 to e3472bd Compare July 3, 2024 16:12
@benrules3 benrules3 force-pushed the benrules1/merge_sdk_fixes branch from e3472bd to d81b9d4 Compare July 3, 2024 16:17
@benrules3 benrules3 requested a review from crclark-cohere July 3, 2024 16:21
@benrules3 benrules3 force-pushed the benrules1/merge_sdk_fixes branch from d81b9d4 to cd26e62 Compare July 3, 2024 16:23
Update from internal SDK improvements

Remove uncalled test (refactored into internal parser)

Fix isort order for formatter
@benrules3 benrules3 force-pushed the benrules1/merge_sdk_fixes branch from cd26e62 to 07293bd Compare July 3, 2024 16:27
@benrules3 benrules3 merged commit 1a378bc into main Jul 3, 2024
3 checks passed
@benrules3 benrules3 deleted the benrules1/merge_sdk_fixes branch July 3, 2024 16:51
@benrules3 benrules3 mentioned this pull request Jul 3, 2024
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.

2 participants