-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Docs: Development setup instructions seem to be incomplete #22732
Comments
I confirm that changing the order of the execution of those 2 steps addressed the typescript related errors. I had gone through the same process with the same errors and then repeated it with the new order and it was seamless. |
Can you please create a PR to fix this in the |
change order of running `npm ci` and `npm link` fix jhipster#22732
document that `npm ci` should be run before `npm link` Fix jhipster#22732
Is the PR worth $100? Can I split it with @CodingSpiderFox who raised the issue? |
@jelharou do not worry, you can request the bug bounty and @CodingSpiderFox also |
Overview of the issue
I wanted to try the latest version of JHipster on my developer machine, so I cloned the repository at the default branch and followed the instructions in CONTRIBUTING.md:
But already the first step failed because the typescript compiler
tsc
could not be found. I was able to fix that by runningnpm i -g typescript
I ran
npm link
again.Then I got an error message that the node type definitions are not available. So I ran
npm i -g @types/node
.Ran
npm link
again but got the same error message.Then I ran
npm ci
. After that, runningnpm link
succeeded.Motivation for or Use Case
Even unexperienced developers should have an easy way to setup jhipster for contributing
Reproduce the error
See description above
Related issues
Suggest a Fix
In the section
Set NPM to use the cloned project
in CONTRIBUTING.md, instruct the user to runnpm ci
beforenpm link
. Currently, these instructions are mentioned in reverse order.JHipster Version(s)
Commit 721e799...
JHipster configuration
Not applicable
Browsers and Operating System
The text was updated successfully, but these errors were encountered: