Skip to content

Conversation

@egdev6
Copy link
Owner

@egdev6 egdev6 commented Jul 13, 2025

This pull request updates the README.md file to improve the installation instructions. It introduces recommendations for managing Node.js versions and simplifies dependency installation steps.

Updates to installation instructions:

  • Added a note suggesting the use of nvm (for UNIX) or nvm-windows (for Windows) to manage multiple Node.js versions, along with a link to the respective repositories. Also included a link to the pnpm installation documentation for users who do not have pnpm installed. ([README.mdR58-R60](https://github.com/egdev6/design-system/pull/59/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R58-R60))
  • Removed the npm install option from the dependency installation section, leaving only the pnpm install command for simplicity and consistency. ([README.mdL68-L75](https://github.com/egdev6/design-system/pull/59/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L68-L75))

@egdev6 egdev6 requested a review from Copilot July 13, 2025 14:23
@egdev6 egdev6 merged commit 83a1623 into main Jul 13, 2025
Copy link
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

This PR enhances the project’s setup documentation by recommending Node.js version managers and streamlining dependency installation.

  • Added instructions and links for using nvm/nvm-windows and pnpm
  • Removed the npm install option, leaving only pnpm install
Comments suppressed due to low confidence (1)

README.md:58

  • [nitpick] Use Markdown link syntax for the repositories and capitalize 'Windows' for consistency, e.g., [nvm](https://github.com/nvm-sh/nvm) and [nvm-windows](https://github.com/coreybutler/nvm-windows).
NOTE: You can install nvm (UNIX - https://github.com/nvm-sh/nvm) or nvm-windows (windows - https://github.com/coreybutler/nvm-windows) to manage multiple node versions.

@egdev6 egdev6 deleted the feature/pnpm-lock branch July 13, 2025 14:24
@netlify
Copy link

netlify bot commented Jul 13, 2025

Deploy Preview for egdev6-design-system failed.

Name Link
🔨 Latest commit dd7f82b
🔍 Latest deploy log https://app.netlify.com/projects/egdev6-design-system/deploys/6873c1750a19500008552ecc

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