Skip to content

Convert JSON.stringify \n to os.EOL - #6610

Merged
ianschmitz merged 2 commits into
react:masterfrom
mikebeaton:minimal-tsconfig-eol-fix
Mar 11, 2019
Merged

Convert JSON.stringify \n to os.EOL#6610
ianschmitz merged 2 commits into
react:masterfrom
mikebeaton:minimal-tsconfig-eol-fix

Conversation

@mikebeaton

Copy link
Copy Markdown
Contributor

Addresses #6566 - intended to operate as discussed

Comment threadpackages/react-scripts/scripts/utils/verifyTypeScriptSetup.js Outdated
@iansuiansu modified the milestones: 2.1.x, 3.0Mar 10, 2019
@ianschmitz
ianschmitz merged commit ed86da9 into react:masterMar 11, 2019
@ianschmitz

Copy link
Copy Markdown
Contributor

Thanks @mikebeaton!

JoviDeCroock added a commit to JoviDeCroock/create-react-app that referenced this pull request Mar 15, 2019
* masterd: (24 commits)
Add TypeScript linting support (react#6513)
Support React Hooks (react#5602) (react#5997)
Support browserslist in @babel/preset-env (react#6608)
Add empty mock for http2 (react#5686)
Add note about npx caching (react#6374)
change named import into default import (react#6625)
Stage files for commit after ejecting (react#5960)
Upgrade dependencies (react#6614)
Make compiler variable const instead of let (react#6621)
Type check JSON files (react#6615)
Change class components to functional components in templates (react#6451)
Convert JSON.stringify \n to os.EOL when writing tsconfig.json (react#6610)
Update html-webpack-plugin (react#6361)
Enable click to go to error in console for TypeScript (react#6502)
Update webpack-dev-server to 3.2.1 (react#6483)
[docs] revert removal of newlines from html (react#6386)
Publish
Prepare 2.1.8 release
Reapply "Speed up TypeScript v2 (react#6406)" (react#6586)
Publish
...
# Conflicts:
#	packages/babel-preset-react-app/create.js
#	packages/react-scripts/scripts/build.js
@locklockBot locked and limited conversation to collaborators Mar 16, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@mikebeaton@ianschmitz@iansu@facebook-github-bot