-
Notifications
You must be signed in to change notification settings - Fork 12
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
Fix doc build instructions #223
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey, thank you for your interest.
I think we should split the build instructions..
let me create another PR and then we'd be able to continue the conversation there.
Edit: Here it's #235
@MustafaJafar with #235 now not merged - is this issue #222 actually still an issue and thus this PR relevant? |
@BigRoy
and I think it's a grey area whether or not to add any explanation. I know that the main audience are the developers. But in CGI world we may have curious people who may know |
I think @johhnry 's point is that the I'm not familiar with NodeJS - so maybe @Innders needs to chime in as to what's sensible in this area? |
I think #283 should solve this issue |
Changelog Description
I opened this issue #222 but realized that the build instructions were outdated. Now
package.json
is in thewebsite
folder so we need to cd into it and install the dependencies.