Skip to content

Use consistent ElevenLabs() call pattern in README.md#594

Open
yxzhao6 wants to merge 1 commit into
elevenlabs:mainfrom
yxzhao6:patch-1
Open

Use consistent ElevenLabs() call pattern in README.md#594
yxzhao6 wants to merge 1 commit into
elevenlabs:mainfrom
yxzhao6:patch-1

Conversation

@yxzhao6
Copy link
Copy Markdown

@yxzhao6 yxzhao6 commented Jul 22, 2025

Lost ~10 minutes confirm that api_key="..." is required to initialize the ElevenLabs() client.

Comment thread README.md
Comment on lines -48 to -52
from dotenv import load_dotenv
from elevenlabs.client import ElevenLabs
from elevenlabs import play

load_dotenv()
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lets keep load_dotenv(), want to fix and then i will approve?

@kraenhansen kraenhansen added the stale PR or issue has not had recent activity and may be closed label Apr 28, 2026
@kraenhansen
Copy link
Copy Markdown
Member

Hey! We're doing a spring cleaning of open PRs. It looks like there are some outstanding requested changes or questions on this PR. Is it still something you'd like to get merged? If so, could you update it when you get a chance? We're marking this as stale for now and will close it in a week if there's no activity.

@kraenhansen
Copy link
Copy Markdown
Member

Also, your branch has merge conflicts and will need a rebase before it can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale PR or issue has not had recent activity and may be closed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants