Skip to content

docs: fix README.md for @github/keytar#18

Open
kushalshit27 wants to merge 1 commit into
github:masterfrom
kushalshit27:fix/readme
Open

docs: fix README.md for @github/keytar#18
kushalshit27 wants to merge 1 commit into
github:masterfrom
kushalshit27:fix/readme

Conversation

@kushalshit27
Copy link
Copy Markdown

Context

The README.md was using the old/wrong package name keytar instead of the scoped name @github/keytar (as declared in package.json).

Changes Made

  • Added both CJS (require) and ESM (import) usage examples under Docs
  • npm install keytarnpm install @github/keytar

Copilot AI review requested due to automatic review settings April 13, 2026 09:30
@kushalshit27 kushalshit27 requested a review from a team as a code owner April 13, 2026 09:30
Copy link
Copy Markdown

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 project README to reference the correct scoped npm package name (@github/keytar) and expands the “Docs” section with both CommonJS and ESM usage examples.

Changes:

  • Update installation instructions to npm install @github/keytar.
  • Update “Supported versions” text to reference @github/keytar.
  • Add both CJS (require) and ESM (import) examples under “Docs”.

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

Comment thread README.md
@andrewjensen
Copy link
Copy Markdown

@williammartin +1 to this pull request! I am planning on adding this as a dependency in a project and noticed the readme had old information.

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.

3 participants