Skip to content

Remove react-scripts type reference on eject #5611

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.

Sign up for GitHub

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

Merged
Timer merged 14 commits into from
Oct 30, 2018
Merged

Remove react-scripts type reference on eject #5611

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
bf782b9
Remove react-scripts type reference on eject
Timer Oct 29, 2018
db33ddd
Check for env file
Timer Oct 29, 2018
c02aa08
Check eject for typescript
Timer Oct 29, 2018
3d39259
Shuffle appTypeDeclarations
Timer Oct 29, 2018
a8dc1a5
Append internal types on eject
Timer Oct 29, 2018
10fd261
Ensure lib is published for types
Timer Oct 29, 2018
37a696d
Adjust comment
Timer Oct 29, 2018
272da12
Don't add a bunch of new lines
Timer Oct 29, 2018
6ee2925
File should exist and not be deleted
Timer Oct 29, 2018
0068ba8
Add debug
Timer Oct 29, 2018
bcd58a3
Set file explicitly
Timer Oct 29, 2018
061062c
Revert "Set file explicitly"
Timer Oct 30, 2018
544b527
Copy file before destroying ourselves
Timer Oct 30, 2018
f32b217
Revert "Add debug"
Timer Oct 30, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Shuffle appTypeDeclarations
  • Loading branch information
@Timer
Timer committed Oct 29, 2018
commit 3d3925927fdd06e6e0ab061249cd76c092633b4a
7 changes: 4 additions & 3 deletions packages/react-scripts/config/paths.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ module.exports = {
appPackageJson: resolveApp('package.json'),
appSrc: resolveApp('src'),
appTsConfig: resolveApp('tsconfig.json'),
appTypeDeclarations: resolveApp('src/react-app-env.d.ts'),
yarnLockFile: resolveApp('yarn.lock'),
testsSetup: resolveModule(resolveApp, 'src/setupTests'),
proxySetup: resolveApp('src/setupProxy.js'),
Expand All @@ -107,7 +106,6 @@ module.exports = {
appPackageJson: resolveApp('package.json'),
appSrc: resolveApp('src'),
appTsConfig: resolveApp('tsconfig.json'),
appTypeDeclarations: resolveApp('src/react-app-env.d.ts'),
yarnLockFile: resolveApp('yarn.lock'),
testsSetup: resolveModule(resolveApp, 'src/setupTests'),
proxySetup: resolveApp('src/setupProxy.js'),
Expand All @@ -117,6 +115,8 @@ module.exports = {
// These properties only exist before ejecting:
ownPath: resolveOwn('.'),
ownNodeModules: resolveOwn('node_modules'), // This is empty on npm 3
appTypeDeclarations: resolveApp('src/react-app-env.d.ts'),
ownTypeDeclarations: resolveOwn('config/react-app.d.ts'),
};

const ownPackageJson = require('../package.json');
Expand All @@ -140,7 +140,6 @@ if (
appPackageJson: resolveOwn('package.json'),
appSrc: resolveOwn('template/src'),
appTsConfig: resolveOwn('template/tsconfig.json'),
appTypeDeclarations: resolveOwn('template/src/react-app-env.d.ts'),
yarnLockFile: resolveOwn('template/yarn.lock'),
testsSetup: resolveModule(resolveOwn, 'template/src/setupTests'),
proxySetup: resolveOwn('template/src/setupProxy.js'),
Expand All @@ -150,6 +149,8 @@ if (
// These properties only exist before ejecting:
ownPath: resolveOwn('.'),
ownNodeModules: resolveOwn('node_modules'),
appTypeDeclarations: resolveOwn('template/src/react-app-env.d.ts'),
ownTypeDeclarations: resolveOwn('config/react-app.d.ts'),
};
}
// @remove-on-eject-end
Expand Down