Skip to content

Clarify installation instructions for OpenKnowledge CLI#533

Closed
robert-inkeep wants to merge 2 commits into
mainfrom
robert-inkeep-patch-10
Closed

Clarify installation instructions for OpenKnowledge CLI#533
robert-inkeep wants to merge 2 commits into
mainfrom
robert-inkeep-patch-10

Conversation

@robert-inkeep

Copy link
Copy Markdown
Contributor

Reword the installation note to describe the result ("run ok from anywhere") instead of implying that the installer adds ok to PATH.

What & why

How this was verified

Checklist

  • Ran bun run check (lint, typecheck, tests) locally
  • Added a changeset (bun run changeset) if this changes behavior
  • Updated docs if this changes a user-facing surface
  • I have read CONTRIBUTING.md and agree to license my contribution under the project's terms (CLA)

After you open this PR

  • A maintainer will review your PR. If you don't hear back within a few business days, a comment to nudge is welcome.
  • When your change is accepted, this PR closes automatically — that's how it merges, and your authorship is preserved.

See CONTRIBUTING.md for details.

Reword the installation note to describe the result ("run `ok` from anywhere") instead of implying that the installer adds `ok` to PATH.
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Your change is now on main. Thanks for contributing!

Because this repository is maintained through an internal mirror, GitHub shows this PR as closed rather than merged, but your commits are on the default branch with your attribution intact. Closing it here to keep things in sync.

inkeep-oss-sync Bot added a commit that referenced this pull request Jul 8, 2026
…nowledge CLI (#2502)

Commits:

1. [0adbbe3](0adbbe3) Clarify installation instructions for OpenKnowledge CLI
   Author: robert-inkeep <123133375+robert-inkeep@users.noreply.github.com>

   Reword the installation note to describe the result ("run `ok` from anywhere") instead of implying that the installer adds `ok` to PATH.
2. [4fa26ef](4fa26ef) newline
   Author: robert-inkeep <123133375+robert-inkeep@users.noreply.github.com>

Co-authored-by: inkeep-oss-sync[bot] <274976938+inkeep-oss-sync[bot]@users.noreply.github.com>
Co-authored-by: robert-inkeep <123133375+robert-inkeep@users.noreply.github.com>
GitOrigin-RevId: 08d4b834babf115c7423d1732eaceba10757eb24
inkeep-oss-sync Bot added a commit that referenced this pull request Jul 8, 2026
…nowledge CLI (#2502)

Commits:

1. [0adbbe3](0adbbe3) Clarify installation instructions for OpenKnowledge CLI
   Author: robert-inkeep <123133375+robert-inkeep@users.noreply.github.com>

   Reword the installation note to describe the result ("run `ok` from anywhere") instead of implying that the installer adds `ok` to PATH.
2. [4fa26ef](4fa26ef) newline
   Author: robert-inkeep <123133375+robert-inkeep@users.noreply.github.com>

Co-authored-by: inkeep-oss-sync[bot] <274976938+inkeep-oss-sync[bot]@users.noreply.github.com>
Co-authored-by: robert-inkeep <123133375+robert-inkeep@users.noreply.github.com>
GitOrigin-RevId: 08d4b834babf115c7423d1732eaceba10757eb24
@inkeep-oss-sync inkeep-oss-sync Bot closed this Jul 8, 2026
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.

1 participant