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

Remove JINA API key from mandatory env vars #303

Merged
izellevy merged 1 commit into from
Feb 26, 2024
Merged

Remove JINA API key from mandatory env vars #303

izellevy merged 1 commit into from
Feb 26, 2024

Conversation

@aulorbe
Copy link
Contributor

@aulorbe aulorbe commented Feb 21, 2024

Problem

JINA API key is not necessary to set up Canopy, so removed from README.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Infrastructure change (CI configs, etc)
  • Non-code change (docs, etc)
  • None of the above: (explain here)

@izellevy izellevy added this pull request to the merge queue Feb 25, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 25, 2024
@izellevy izellevy added this pull request to the merge queue Feb 26, 2024
Merged via the queue into pinecone-io:main with commit 8dfa859 Feb 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

3 more reviewers

@izellevy izellevy izellevy approved these changes

@junefish junefish junefish approved these changes

@zeke-emerson zeke-emerson zeke-emerson approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@aulorbe @izellevy @junefish @zeke-emerson