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

[config][cli] Remove classic manifest types #24054

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

wschurman
Copy link
Member

@wschurman wschurman commented Aug 22, 2023

Why

Remove classic manifest types from @expo/config and the CLI. This is a breaking change as it removes the --force-manifest-type flag from the CLI (which was planned).

How

Remove types. Remove classic manifest serving from expo start.

Test Plan

Run all tests.

Checklist

@wschurman wschurman force-pushed the @wschurman/reorganize-types-2 branch from 06c2a8c to 32b071a Compare August 22, 2023 00:54
@wschurman wschurman force-pushed the @wschurman/reorganize-types-3 branch 2 times, most recently from 733c41f to b88f433 Compare August 22, 2023 18:06
@wschurman wschurman changed the title [config] Remove classic manifest types [config][cli] Remove classic manifest types Aug 22, 2023
@wschurman wschurman marked this pull request as ready for review August 22, 2023 18:09
@wschurman wschurman requested review from douglowder and removed request for Simek and amandeepmittal August 22, 2023 18:09
@wschurman wschurman force-pushed the @wschurman/reorganize-types-3 branch from b88f433 to 32811bb Compare August 22, 2023 18:45
@wschurman
Copy link
Member Author

Failing test is due to issue in main that was fixed after the base commit of this stack. This is ready for review.

@wschurman wschurman force-pushed the @wschurman/reorganize-types-2 branch from 32b071a to c21dc72 Compare August 23, 2023 16:23
@wschurman wschurman force-pushed the @wschurman/reorganize-types-2 branch from c21dc72 to fdf61c1 Compare August 23, 2023 16:27
Base automatically changed from @wschurman/reorganize-types-2 to main August 23, 2023 16:29
@wschurman wschurman force-pushed the @wschurman/reorganize-types-3 branch from 32811bb to 29d6234 Compare August 23, 2023 16:30
@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Aug 23, 2023
@wschurman wschurman force-pushed the @wschurman/reorganize-types-3 branch from 29d6234 to 4ff5c30 Compare August 23, 2023 16:33
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Aug 23, 2023
@wschurman wschurman merged commit 9ba03fb into main Aug 23, 2023
16 checks passed
@wschurman wschurman deleted the @wschurman/reorganize-types-3 branch August 23, 2023 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants