Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

react-solution not spinning up #56

Open
brianamarie opened this issue Dec 30, 2019 · 3 comments
Open

react-solution not spinning up #56

brianamarie opened this issue Dec 30, 2019 · 3 comments

Comments

@brianamarie
Copy link
Contributor

brianamarie commented Dec 30, 2019

Bug Report

Current behavior
The repository used as a "solution", https://github.com/githubtraining/react-solution/, will not spin up correctly locally with npm start.

Reproduction
Steps to reproduce the behavior in the course:

  1. Clone https://github.com/githubtraining/react-solution/
  2. CD into the repo
  3. Run npm start
  4. See error

Expected behavior
I would expect the solution repository to be able to work immediately upon clone

Additional context
I found this out when testing a dependabot PR - I wanted to see that things worked as they would on master and found that master doesn't work either.

Whatever the solution is, it should probably also be applied to https://github.com/githubtraining/react-template/.

cc @hectorsector @partyshah

@brianamarie brianamarie added the bug Something isn't working label Dec 30, 2019
@Sid-ah
Copy link

Sid-ah commented May 10, 2020

You’re missing a step which is to run npm install. It will install the required libraries and one of those libraries is react-scripts, which is responsible for spinning up the project.

@mattdavis0351
Copy link

mattdavis0351 commented Jul 30, 2020

@brianamarie we wouldn't want to add the dependencies to the template repo. That's against best practice. @hectorsector what are your thoughts on adding dependencies to the template repo? If you are in the camp of not doing so (the better camp 😉 ) then this issue can be moved along to done in the triage board.

The README in https://github.com/githubtraining/react-solution/ is the default create-react-app README. The solution to this would be to edit that README to include proper steps for running the app. Which is a different work item.

Edit:

I have updated the README to include an npm install step.

@hectorsector
Copy link
Member

The solution isn't the template repo though, right?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants