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

Remove JINA API key from mandatory env vars #303

Merged
merged 1 commit into from
Feb 26, 2024
Merged

Remove JINA API key from mandatory env vars #303

merged 1 commit into from
Feb 26, 2024

Conversation

aulorbe
Copy link
Collaborator

@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
7 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

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

@junefish junefish junefish approved these changes

@izellevy izellevy izellevy approved these changes

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