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
File should exist and not be deleted
  • Loading branch information
@Timer
Timer committed Oct 29, 2018
commit 6ee2925c129c757ed50ee38d4dff6d0032b1bcf0
4 changes: 2 additions & 2 deletions tasks/e2e-installs.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,8 +183,8 @@ CI=true yarn test
# Eject...
echo yes | npm run eject

# Ensure env file was removed
! exists src/react-app-env.d.ts
# Ensure env file still exists
exists src/react-app-env.d.ts

# Check that the TypeScript template passes ejected smoke tests, build, and normal tests
yarn start --smoke-test
Expand Down