-
Notifications
You must be signed in to change notification settings - Fork 5
docs: add Linux CLI terminal installation instructions and clean up outdated steps #147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The PR is failing due to a lint.mjs shellcheck step that uses find -regextype, which isn’t supported on macOS or the default GitHub runner. No .sh files were added or changed in this PR, so this isn’t related to the actual changes. @CMCDragonkai @aryanjassal |
I guess a platform-agnostic way to do this could be this? I'll have to dig into this later.
|
Actually nevermind, that is not the issue. We use nix wrappers in the CI so that wouldn't be an issue. If we look at the error, this is buried between the error somewhere.
It says that prettier found some code issues so you gotta fix that instead. Try running |
3631c74
to
49c4125
Compare
b5d4397
to
6f4e208
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's mostly good, just a minor update
Description
This PR updates the Polykey CLI installation documentation to:
This resolves confusion around why subsequent Polykey commands don’t work in the same terminal window after starting the agent, addressing a critical usability gap in the existing CLI documentation.
Issues Fixed
Tasks
Final checklist