Skip to content

README.md: remove DownloadAsync() example#911

Merged
jpdillingham merged 1 commit intojpdillingham:masterfrom
slook:patch-1
Mar 29, 2026
Merged

README.md: remove DownloadAsync() example#911
jpdillingham merged 1 commit intojpdillingham:masterfrom
slook:patch-1

Conversation

@slook
Copy link
Copy Markdown
Contributor

@slook slook commented Mar 26, 2026

  • Removed: "Download a file" section; it's not really needed here, nor is it a prime example of the library's capabilities
  • Changed: Corrected a grammatical mistake in the "Search for something" section

Suggest a different function could be added here instead as a better example, such as responding to a search request or sending a chat message.

@jpdillingham
Copy link
Copy Markdown
Owner

Thanks. I think I'll strip all examples for now.

@jpdillingham jpdillingham merged commit 4dd1b69 into jpdillingham:master Mar 29, 2026
0 of 2 checks passed
@slook
Copy link
Copy Markdown
Contributor Author

slook commented Mar 29, 2026

It'll be good to steer the docs towards the library having the intent of making a fully-featured client rather than it being seen as a handy shortcut to make bare-bones partial implementation for some slop bot tool!

@jpdillingham
Copy link
Copy Markdown
Owner

I agree, I just can't spare the time to write them up. In the meantime if someone really wants they can spend the tokens to analyze the code :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants