Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

add: support for custom API paths #53

Merged
merged 2 commits into from
Apr 18, 2023

Conversation

JettChenT
Copy link
Contributor

This allows users to use custom openAI API paths by setting the environment variable(config) custom_api_url to their custom API url address.

#51

@JettChenT JettChenT marked this pull request as draft April 14, 2023 11:45
@JettChenT JettChenT force-pushed the feat-custom-api branch 2 times, most recently from 417481d to d6ffe94 Compare April 14, 2023 11:55
@JettChenT JettChenT marked this pull request as ready for review April 14, 2023 11:57
@chrislemke
Copy link
Owner

chrislemke commented Apr 14, 2023

Hey @JettChenT!

  1. Could you provide an example URL and
  2. Could you please merge it to develop and not main so I could test it a bit?
  3. Please add some information about this to the README.md so user understand how this feature works

fix: typo

fix: typo

docs: add alfred workflow config and readme descriptions
@JettChenT JettChenT changed the base branch from main to develop April 14, 2023 15:44
@JettChenT
Copy link
Contributor Author

JettChenT commented Apr 14, 2023

Hi, I'm wondering if I can get some help regarding the pre commit CI? Running the pre-commit hook locally skips both the black check and the docformatter check.

@JettChenT
Copy link
Contributor Author

Hey @JettChenT!

  1. Could you provide an example URL and
  2. Could you please merge it to develop and not main so I could test it a bit?
  3. Please add some information about this to the README.md so user understand how this feature works

The example url is https://closeai.deno.dev/v1, which added in to the README.md

@chrislemke
Copy link
Owner

Looks like some tests/checks are failing. You can use: https://pre-commit.com to solve them or do it manually.

Thanks!

@chrislemke
Copy link
Owner

@JettChenT!
Sorry for the late response. Let me have a look. I can do it.

@chrislemke chrislemke merged commit d03fa30 into chrislemke:develop Apr 18, 2023
@chrislemke
Copy link
Owner

@JettChenT I just merged your stuff. Thanks for contributing!

@JettChenT
Copy link
Contributor Author

Thanks! Sorry I haven't been able to respond the past few days.

@JettChenT JettChenT deleted the feat-custom-api branch April 20, 2023 00:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants