Skip to content
This repository was archived by the owner on Apr 30, 2023. It is now read-only.

Update template for React 18 and React-Redux v8 #53

Merged
merged 7 commits into from
Apr 13, 2022
Merged

Update template for React 18 and React-Redux v8 #53

merged 7 commits into from
Apr 13, 2022

Conversation

souzaramon
Copy link
Contributor

#52

@souzaramon souzaramon marked this pull request as ready for review April 13, 2022 02:17
@souzaramon
Copy link
Contributor Author

I think that's it

@markerikson
Copy link
Contributor

markerikson commented Apr 13, 2022

Hmm. Looks like our Github workflow needs to be updated to set NODE_ENV=test ?

Run npm run lint

> [email protected] lint /home/runner/work/cra-template-redux/cra-template-redux
> eslint './template/src/**/*.js'

/home/runner/work/cra-template-redux/cra-template-redux/template/src/App.js
  0:0  error  Parsing error: [BABEL] /home/runner/work/cra-template-redux/cra-template-redux/template/src/App.js: Using `babel-preset-react-app` requires that you specify `NODE_ENV` or `BABEL_ENV` environment variables. Valid values are "development", "test", and "production". Instead, received: undefined. (While processing: "/home/runner/work/cra-template-redux/cra-template-redux/node_modules/babel-preset-react-app/index.js")

Can I have you tweak that and get the check passing?

@souzaramon
Copy link
Contributor Author

souzaramon commented Apr 13, 2022

Hmm. Looks like our Github workflow needs to be updated to set NODE_ENV=test ?

Run npm run lint

> [email protected] lint /home/runner/work/cra-template-redux/cra-template-redux
> eslint './template/src/**/*.js'

/home/runner/work/cra-template-redux/cra-template-redux/template/src/App.js
  0:0  error  Parsing error: [BABEL] /home/runner/work/cra-template-redux/cra-template-redux/template/src/App.js: Using `babel-preset-react-app` requires that you specify `NODE_ENV` or `BABEL_ENV` environment variables. Valid values are "development", "test", and "production". Instead, received: undefined. (While processing: "/home/runner/work/cra-template-redux/cra-template-redux/node_modules/babel-preset-react-app/index.js")

Can I have you tweak that and get the check passing?

I add NODE_ENV=test in the github flow and in dev I had to add NODE_ENV=dev too, that is a issue about this here.

facebook/create-react-app#2377

And thinking about windows env, cross-env should be the simplier solution.

@markerikson
Copy link
Contributor

Looks good!

@markerikson markerikson merged commit 409239e into reduxjs:master Apr 13, 2022
@souzaramon souzaramon deleted the feat/react-18-and-redux-v8 branch April 13, 2022 18:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants