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

Commit

Permalink
fix: prevent netlify dev from attempting to recursively call itself
Browse files Browse the repository at this point in the history
The CLI detects different framework commands in package.json, and will throw
if it detects `netlify dev` in the default script while running `netlify dev`
in order to avoid recursively calling itself.

By removing the `netlify dev` from the `start` and `dev` scripts and adjusting
the CLI logic to recognize `netlify dev:exec` as distinct from `netlify:dev`,
we can get this project working as expected.

Also updates package.json dependencies.

see also:
#26
netlify/cli#1798
  • Loading branch information
ctjlewis committed Jan 24, 2021
1 parent 159b95d commit ef53022
Show file tree
Hide file tree
Showing 5 changed files with 22,700 additions and 7,703 deletions.
6 changes: 6 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# See https://help.github.com/ignore-files/ for more about ignoring files.

# .env
.env

# eslint cache
.eslintcache

# dependencies
/node_modules

Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,10 +96,10 @@ Click the [Deploy to Netlify Button](https://app.netlify.com/start/deploy?reposi
npm run bootstrap
```

10. Run project locally
10. Start developing!

```bash
npm start
netlify dev
```

## TLDR; Quick Deploy
Expand Down
4 changes: 2 additions & 2 deletions netlify.toml
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,5 @@
publish = "build"

[dev]
# Local dev command. A.k.a npm start
command = "react-scripts start"
# Call react-scripts start
command = "npm start"
Loading

0 comments on commit ef53022

Please sign in to comment.