History log of /expo/packages/@expo/cli/src/start/server/metro/__tests__/instantiateMetro.test.ts (Results 1 – 1 of 1)
Revision Date Author Comments
# 0f61c876 25-Jan-2023 Cedric van Putten <[email protected]>

fix(cli): simplify Metro watch mode and communicate when disabled (#20939)

# Why

Fixes ENG-7089

# How

Because this option is only exposed through `Metro.runServer`, I added
the override in

fix(cli): simplify Metro watch mode and communicate when disabled (#20939)

# Why

Fixes ENG-7089

# How

Because this option is only exposed through `Metro.runServer`, I added
the override in `@expo/cli`. Alternative, we could add a custom Metro
config option to move this to the config itself, however, that would
create a significant diversion of the normal Metro config.

> **Note**
> We could add a Metro version check and disable the warning, but I
think it's safe to skip that and leave it in for older Metro versions.
It's not really synced, but if the message pops up, it's very very
likely Metro actually is running without watch.

# Test Plan

See added tests.

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
This is required for changes to Expo modules.
-->

- [ ] Documentation is up to date to reflect these changes (eg:
https://docs.expo.dev and README.md).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
- [ ] This diff will work correctly for `expo prebuild` & EAS Build (eg:
updated a module plugin).

show more ...