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

[README] - Fix misleading phrasing #146

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

igiloh-pinecone
Copy link
Contributor

Problem

The "considerations" section had some misleading \ incorrect phrasing.

Type of Change

  • Non-code change (docs, etc)

The "considerations" section had some misleading \ incorrect phrasing.
@@ -220,4 +221,4 @@ To run the canopy server for production, please run:
gunicorn canopy_cli.app:app --worker-class uvicorn.workers.UvicornWorker --bind 0.0.0.0:8000 --workers <number of desired worker processes>
```

The server interacts with services like Pinecone and OpenAI using your own authentication credentials. When deploying the server on a public web hosting provider, it is recommended to enable an authentication mechanism, so that your server would only take requests from authenticated users.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

No changes here, ignore.

Copy link
Contributor

@miararoy miararoy left a comment

Choose a reason for hiding this comment

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

Lg

@igiloh-pinecone igiloh-pinecone merged commit e3cd282 into main Nov 6, 2023
4 checks passed
@igiloh-pinecone igiloh-pinecone deleted the igiloh-pinecone-patch-1 branch November 6, 2023 16:54
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