Skip to content

Docs: re-org and improve getting started docs#290

Merged
yanyongyu merged 4 commits intomasterfrom
docs/reorg-usage-start
Apr 4, 2026
Merged

Docs: re-org and improve getting started docs#290
yanyongyu merged 4 commits intomasterfrom
docs/reorg-usage-start

Conversation

@yanyongyu
Copy link
Copy Markdown
Owner

resolve #285

@yanyongyu yanyongyu added the documentation Improvements or additions to documentation label Mar 28, 2026
repo: FullRepository = resp.parsed_data
```

!!! note
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure why you included this, I don't think someone is likely to encounter this.
Also this is a general python error not something specific to githubkit.

Copy link
Copy Markdown
Contributor

@galshi galshi Apr 3, 2026

Choose a reason for hiding this comment

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

Also, not 100% it's relevant but if someone has a use case where he creates a lot of clients (let's say github app with oauth) you would want a way to open an unknown amount of contexts, although it's also useful for people who just need to open a lot of contexts instead of doing with a, b, c, d...:.

I use something like this:

async with contextlib.AsyncExitStack() as stack:
    for client in client_list:
        await stack.enter_async_context(client)

Might be useful for other people too.

yanyongyu and others added 2 commits April 4, 2026 11:35
Co-authored-by: galshi <59773096+galshi@users.noreply.github.com>
@yanyongyu yanyongyu merged commit dc7cb17 into master Apr 4, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Native memory leak when not using async with GitHub

2 participants