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

Docs: Development setup instructions seem to be incomplete #22732

Closed
1 task done
spotlesscoder opened this issue Jul 1, 2023 · 4 comments · Fixed by #22808
Closed
1 task done

Docs: Development setup instructions seem to be incomplete #22732

spotlesscoder opened this issue Jul 1, 2023 · 4 comments · Fixed by #22808
Labels
area: documentation:books: $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ $100 https://www.jhipster.tech/bug-bounties/
Milestone

Comments

@spotlesscoder
Copy link
Contributor

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:

Run npm link and then run npm ci

But already the first step failed because the typescript compiler tsc could not be found. I was able to fix that by running npm 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, running npm 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 run npm ci before npm link. Currently, these instructions are mentioned in reverse order.

JHipster Version(s)

Commit 721e799...

JHipster configuration

Not applicable

Browsers and Operating System
  • Checking this box is mandatory (this is just to show you read everything)
@spotlesscoder spotlesscoder changed the title Development setup instructions seem to be incomplete Docs: Development setup instructions seem to be incomplete Jul 2, 2023
@jelharou
Copy link
Contributor

jelharou commented Jul 3, 2023

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.

@mraible
Copy link
Contributor

mraible commented Jul 8, 2023

Can you please create a PR to fix this in the CONTRIBUTING.md document? I'll add a bug bounty to help motivate.

@mraible mraible added $100 https://www.jhipster.tech/bug-bounties/ $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ labels Jul 8, 2023
jelharou added a commit to jelharou/generator-jhipster that referenced this issue Jul 8, 2023
change order of running `npm ci` and `npm link`

fix jhipster#22732
jelharou added a commit to jelharou/generator-jhipster that referenced this issue Jul 8, 2023
document that `npm ci` should be run before `npm link`

Fix jhipster#22732
@jelharou
Copy link
Contributor

jelharou commented Jul 9, 2023

Can you please create a PR to fix this in the CONTRIBUTING.md document? I'll add a bug bounty to help motivate.

Is the PR worth $100? Can I split it with @CodingSpiderFox who raised the issue?

@DanielFran
Copy link
Member

@jelharou do not worry, you can request the bug bounty and @CodingSpiderFox also

@DanielFran DanielFran added this to the 8.0.0-beta.2 milestone Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: documentation:books: $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ $100 https://www.jhipster.tech/bug-bounties/
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants