Skip to content

Conversation

@puckowski
Copy link

Update contributing to Less.js documentation to:

  • reflect pnpm usage
  • reflect the fact that PhantomJS is no longer required
  • update grunt command options
  • fix some Less.js GitHub links

Tried to adhere to the original page contents while providing correct updated information.

* Update contributing to Less.js documentation to reflect pnpm usage and
  reflect the fact that PhantomJS is no longer required. Update grunt
  command options. Fix some Less.js GitHub links.
If you look in the libs folder you will see `less`, `less-node`, `less-browser`. The `less` folder is pure javascript with no environment
specifics. if you require `less/libs/less`, you get a function that takes an environment object and an array of file managers. The file
managers are the same file managers that can also be written as a plugin.
If you look in the libs folder you will see `less`, `less-node`, `less-browser`. The `less` folder is pure javascript with no environment specifics. if you require `less/libs/less`, you get a function that takes an environment object and an array of file managers. The file managers are the same file managers that can also be written as a plugin.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: "if you require" should start with an upper case "I"

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for reviewing. Should be corrected in latest code now.

* Fix documentation spelling by switching to a capital 'i'.
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