Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CONTRIBUTING.md and fix dependencies #410

Merged

Conversation

puresick
Copy link
Contributor

@puresick puresick commented Jun 3, 2023

Working on Issue 406, I noticed a few pieces of information missing to properly setup the project to be able to run it locally.

This PR updates the following things:

  • Missing information about the development setup has been added to CONTRIBUTING.md which are needed to setup the project and get it running
    • To determine the latest Node.js version the current used vuepress version is working with, I tested all the releases listed on the Node.js project page
    • Added .node-version configuration file for Node.js version managers to ensure correct version is used
  • Missing project dependency SockJS-client has been added to package.json and package-lock.json

Copy link
Contributor

@Confectrician Confectrician left a comment

Choose a reason for hiding this comment

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

Thanks. Just some minor grammar thing i will contribute on my own. :)

CONTRIBUTING.md Outdated Show resolved Hide resolved
Signed-off-by: Jerome Luckenbach <github@luckenba.ch>
@Confectrician Confectrician merged commit 1c19c91 into openhab:main Jul 1, 2023
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