Skip to content

Update installation command for runpod-flash#335

Merged
deanq merged 3 commits into
runpod:mainfrom
rand:patch-1
Jun 3, 2026
Merged

Update installation command for runpod-flash#335
deanq merged 3 commits into
runpod:mainfrom
rand:patch-1

Conversation

@rand
Copy link
Copy Markdown
Contributor

@rand rand commented May 8, 2026

Updating from uv add to uv tool install so that subsequent directions in the README flow more easily.

Updating from `uv add` to `uv tool install` so that subsequent directions in the README flow more easily.
@promptless
Copy link
Copy Markdown

promptless Bot commented May 8, 2026

Promptless prepared a documentation update related to this change.

Triggered by PR #335

Updated flash/overview.mdx to change the uv installation command from uv add runpod-flash to uv tool install runpod-flash, aligning the docs with this SDK change.

Review: Update uv installation command

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the README installation guidance for Flash to favor installing the CLI via uv tool install.

Changes:

  • Replaces the uv installation command from uv add runpod-flash to uv tool install runpod-flash.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Copy link
Copy Markdown
Member

@deanq deanq left a comment

Choose a reason for hiding this comment

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

@rand I have just tried this out in a fresh test project. It's not working. 401 each time. I'll check around what's wrong, but we can't just change this and end up breaking everyone's workflow.

@rand
Copy link
Copy Markdown
Contributor Author

rand commented Jun 2, 2026

@deanq the change was intended to be specific to the workflow in the docs, which were immediately followed by having the user invoke flash as a cli tool. It still needs to be added as a project (uv add) for use within a Python environment.

@deanq deanq requested review from KAJdev, jhcipar and runpod-Henrik June 3, 2026 02:10
@deanq
Copy link
Copy Markdown
Member

deanq commented Jun 3, 2026

There was something wrong with my environment that caused the errors. I have it setup right now with uv tool install runpod-flash and makes it easy to run CLI without having to deal with venv. This was a constant speed bump during our usability tests. Thanks!

Copy link
Copy Markdown
Member

@deanq deanq left a comment

Choose a reason for hiding this comment

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

works for me. @runpod-Henrik ?

@deanq deanq merged commit f45a05a into runpod:main Jun 3, 2026
5 checks passed
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.

4 participants