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

Fix lint failure for openai binding #3000

Merged
merged 7 commits into from
Jul 21, 2023

Conversation

shivam-51
Copy link
Contributor

@shivam-51 shivam-51 commented Jul 20, 2023

Description

This PR fixes some linting errors that is causing dapr/dapr#6678 to fail.
The OpenAI space is rapidly getting updates so errors like this can be expected.

Issue reference

Please reference the issue this PR will close: #2716

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Signed-off-by: Shivam Kumar Singh <[email protected]>
@shivam-51 shivam-51 requested review from a team as code owners July 20, 2023 05:11
@shivam-51
Copy link
Contributor Author

@ItalyPaleAle Can you take a look. This should fix the lining errors caused in dapr/dapr PR.

ItalyPaleAle
ItalyPaleAle previously approved these changes Jul 20, 2023
@ItalyPaleAle
Copy link
Contributor

The OpenAI space is rapidly getting updates so errors like this can be expected.

Perhaps we should have waited for at least a versioned release. Looks like we are still using development builds.

@ItalyPaleAle
Copy link
Contributor

This may be relevant too: Azure/azure-sdk-for-go#21214

Signed-off-by: Shivam Kumar Singh <[email protected]>
auto-merge was automatically disabled July 20, 2023 06:51

Head branch was pushed to by a user without write access

@shivam-51
Copy link
Contributor Author

@ItalyPaleAle Please have a look.

Signed-off-by: ItalyPaleAle <[email protected]>
Signed-off-by: ItalyPaleAle <[email protected]>
Signed-off-by: ItalyPaleAle <[email protected]>
Copy link
Contributor

@ItalyPaleAle ItalyPaleAle left a comment

Choose a reason for hiding this comment

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

Thanks!

@ItalyPaleAle ItalyPaleAle added this pull request to the merge queue Jul 21, 2023
Merged via the queue into dapr:master with commit 9957d69 Jul 21, 2023
82 of 84 checks passed
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.

[azure openai] support chatgpt as binding
2 participants