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

feat(cli): prevent showing logs for source maps #25830

Merged
merged 2 commits into from
Dec 9, 2023

Conversation

EvanBacon
Copy link
Contributor

Why

Screenshot 2023-12-08 at 8 18 34 PM

How

Detect if a bundling event is for a map, if so, hide it from the logger.

Test Plan

  • Start dev server.
  • Add a bunch of modules.
  • Comment them out / back in.
  • Close/open inspector to trigger all the HMR sourcemaps being refetched.
  • There shouldn't be any additional request logs in the terminal.

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Dec 9, 2023
Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Dec 9, 2023
@EvanBacon EvanBacon merged commit 838f1cd into main Dec 9, 2023
6 checks passed
@EvanBacon EvanBacon deleted the @evanbacon/cli/prevent-showing-logs-for-sourcemaps branch December 9, 2023 04:44
onizam95 pushed a commit to onizam95/expo-av-drm that referenced this pull request Jan 15, 2024
# Why

<img width="904" alt="Screenshot 2023-12-08 at 8 18 34 PM"
src="https://github.com/expo/expo/assets/9664363/af868d0b-6918-4bbc-b557-d2269ee935e0">

<!--
Please describe the motivation for this PR, and link to relevant GitHub
issues, forums posts, or feature requests.
-->

# How

Detect if a bundling event is for a map, if so, hide it from the logger.

<!--
How did you build this feature or fix this bug and why?
-->

# Test Plan

- Start dev server.
- Add a bunch of modules.
- Comment them out / back in.
- Close/open inspector to trigger all the HMR sourcemaps being
refetched.
- There shouldn't be any additional request logs in the terminal.

---------

Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: fingerprint changed bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants