Skip to content
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.

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

Add ./dist to the existing list inside the tsconfig.base.json #22314

Open
dmitryusikriseapps opened this issue Apr 28, 2023 · 8 comments
Open
Labels
needs review Issue is ready to be reviewed by a maintainer

Comments

@dmitryusikriseapps
Copy link

Minimal reproducible example

https://pastebin.com/yfsyEMrM

Summary

As soon as the eas update command has been run then the dist folder is created inside the current project. Then, if we run the tsc command, then it will fail (see for reference -> microsoft/TypeScript#50675 (comment)).

We can specify a custom exclude option inside the tsconfig.json, but it'll overwrite the base config.

The suggestion is to add the dist folder to the existing list inside the tsconfig.base.json.

Environment

expo-env-info 1.0.5 environment info:
System:
OS: macOS 13.3.1
Shell: 5.9 - /bin/zsh
Binaries:
Node: 16.19.1 - ~/.nvm/versions/node/v16.19.1/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.19.3 - ~/.nvm/versions/node/v16.19.1/bin/npm
Watchman: 2023.03.20.00 - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.12.0 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK:
API Levels: 29, 30, 31, 32, 33, 33
Build Tools: 30.0.2, 33.0.2
System Images: android-33 | Google Play ARM 64 v8a
IDEs:
Android Studio: 2022.1 AI-221.6008.13.2211.9619390
Xcode: 14.3/14E222b - /usr/bin/xcodebuild
npmPackages:
babel-preset-expo: ^9.3.0 => 9.3.0
expo: ^48.0.0 => 48.0.7
react: 18.2.0 => 18.2.0
react-native: 0.71.6 => 0.71.6
npmGlobalPackages:
eas-cli: 3.10.0
expo-cli: 6.3.2
Expo Workflow: managed

@dmitryusikriseapps dmitryusikriseapps added the needs validation Issue needs to be validated label Apr 28, 2023
@expo-bot expo-bot added needs review Issue is ready to be reviewed by a maintainer and removed needs validation Issue needs to be validated labels Apr 28, 2023
@danrivett
Copy link

Any update on this bug as I've run into this and seems like a simple fix if someone takes a look.

@LinusU
Copy link
Contributor

LinusU commented May 24, 2023

Ran into this today as well, would be great to have dist ignored by default...

@macksal
Copy link

macksal commented Aug 10, 2023

Hit this today too, seems like a straightforward fix that would save others some time

@danrivett
Copy link

Bump. Again ran into this today. It appears to be a 2 minute fix, so really hoping someone from Expo will pick this up soon.

Copy link
Contributor

This issue is stale because it has been open for 90 days with no activity. If there is no activity in the next 7 days, the issue will be closed.

@github-actions github-actions bot added the stale label Nov 20, 2023
@danrivett
Copy link

This really needs to be fixed. It's such a simple change unless I've misunderstood something.

@github-actions github-actions bot removed the stale label Nov 20, 2023
Copy link
Contributor

This issue is stale because it has been open for 90 days with no activity. If there is no activity in the next 7 days, the issue will be closed.

@github-actions github-actions bot added the stale label Feb 18, 2024
@danrivett
Copy link

Still an issue.

@github-actions github-actions bot removed the stale label Feb 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs review Issue is ready to be reviewed by a maintainer
Projects
None yet
Development

No branches or pull requests

5 participants