Skip to content

Enable CI to check for formatting #111

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.

Sign up for GitHub

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

Jump to bottom
Closed
hallacy opened this issue Jul 18, 2022 · 0 comments
Closed

Enable CI to check for formatting #111

hallacy opened this issue Jul 18, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@hallacy
Copy link
Collaborator

hallacy commented Jul 18, 2022

We should enable consistent formatting and require PRs to have that formatting
@hallacy hallacy added the enhancement New feature or request label Jul 18, 2022
@Yadnyawalkya Yadnyawalkya mentioned this issue Oct 9, 2022
Closed
techthiyanes pushed a commit to techthiyanes/openai-python that referenced this issue Oct 2, 2023
@hallacy
Version 11.6 diff (openai#66)
f4be8f2 
* Hallacy/pickleable exceptions (openai#109)

* Exceptions in python are hard to pickle.  Make Exceptions pickleable

* Remove debug

* Added tests

* nit

* Change embeddings call to openai.Embedding.create (openai#110)

* Change embeddings call to openai.Embedding.create

* And update model names

* Untweak notebook

* Support encoded embeddings response (openai#111)

* Support encoded embeddings response

* Removed imports

* Add a couple of comments

* Typo

* Remove CA bundle (openai#112)

* Remove CA bundle

* Removed some references
cgayapr pushed a commit to cgayapr/openai-python that referenced this issue Dec 14, 2024
@hallacy
Version 11.6 diff (openai#66)
f2d1ce7 
* Hallacy/pickleable exceptions (openai#109)

* Exceptions in python are hard to pickle.  Make Exceptions pickleable

* Remove debug

* Added tests

* nit

* Change embeddings call to openai.Embedding.create (openai#110)

* Change embeddings call to openai.Embedding.create

* And update model names

* Untweak notebook

* Support encoded embeddings response (openai#111)

* Support encoded embeddings response

* Removed imports

* Add a couple of comments

* Typo

* Remove CA bundle (openai#112)

* Remove CA bundle

* Removed some references
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Setting up CI to check for formatting Yadnyawalkya/openai-python
2 participants
@rattrayalex @hallacy