Skip to content

docs(readme): replace Bun-only instructions with npm#17

Merged
nicomiguelino merged 3 commits intomainfrom
docs/update-readme-npm-support
May 2, 2026
Merged

docs(readme): replace Bun-only instructions with npm#17
nicomiguelino merged 3 commits intomainfrom
docs/update-readme-npm-support

Conversation

@nicomiguelino
Copy link
Copy Markdown
Collaborator

Summary

  • Remove Bun-required callout; package now works with npm and Node.js
  • Add npm as primary installation method, Bun as alternative
  • Replace bun commands with npm equivalents throughout
  • Fix components doc link from master to main

- Remove Bun-required callout; package now works with npm and Node.js
- Add npm as primary installation method, Bun as alternative
- Replace bun link/unlink with npm link/unlink in local dev setup
- Replace bun run commands with npm run equivalents
- Fix components doc link from master to main branch
@nicomiguelino nicomiguelino self-assigned this May 2, 2026
@nicomiguelino nicomiguelino marked this pull request as ready for review May 2, 2026 04:43
@nicomiguelino nicomiguelino requested a review from Copilot May 2, 2026 04:43
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 project README to reflect that @screenly/edge-apps can be installed and used primarily with npm/Node.js (with Bun as an alternative), aligning the docs with the current package/tooling.

Changes:

  • Replace Bun-only installation/setup instructions with npm-first equivalents.
  • Update CLI usage examples to use npm run ... commands.
  • Fix the components documentation link to reference main instead of master.

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

Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md
…version

- Add Node.js >= 20.6.0 requirement note near installation
- Add npm install and npm run build steps before npm link
- Replace npm unlink with npm install to restore registry version
@nicomiguelino nicomiguelino merged commit 200e599 into main May 2, 2026
4 checks passed
@nicomiguelino nicomiguelino deleted the docs/update-readme-npm-support branch May 2, 2026 05:14
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