Skip to content

Commit

Permalink
chore(expo-router): reduce usage of expo-constants (#26834)
Browse files Browse the repository at this point in the history
- reduce chance of issues like this
expo/router#918
- Using globals directly is the future. A lot less bundling and
resolving.

Manually ran this in a dev client, web, and expo go for ios.

---------

Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
  • Loading branch information
EvanBacon and expo-bot committed Feb 1, 2024
1 parent dc6416e commit dd09948
Show file tree
Hide file tree
Showing 23 changed files with 58 additions and 103 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@ function getEnvironmentForBuildDetails(bundleDetails?: BundleDetails | null): st
// Expo CLI will pass `customTransformOptions.environment = 'node'` when bundling for the server.
const env = bundleDetails?.customTransformOptions?.environment ?? null;
if (env === 'node') {
return `${chalk.bold(' λ ')} `;
return chalk.bold('λ') + ' ';
}

return '';
Expand Down
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`_getBundleStatusMessage should format standard progress for a server invocation 1`] = `" λ ./index.js ▓▓▓▓▓▓▓▓░░░░░░░░ 50.0% ( 50/100)"`;
exports[`_getBundleStatusMessage should format standard progress for a server invocation 1`] = `"λ ./index.js ▓▓▓▓▓▓▓▓░░░░░░░░ 50.0% ( 50/100)"`;
2 changes: 2 additions & 0 deletions packages/expo-router/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@

### 💡 Others

- Reduce usage of `expo-constants`. ([#26834](https://github.com/expo/expo/pull/26834) by [@EvanBacon](https://github.com/EvanBacon))

## 3.4.6 — 2024-01-26

_This version does not introduce any user-facing changes._
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions packages/expo-router/build/fork/extractPathFromURL.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/expo-router/build/fork/extractPathFromURL.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion packages/expo-router/build/head/ExpoHeadModule.d.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/expo-router/build/head/ExpoHeadModule.d.ts.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 6 additions & 1 deletion packages/expo-router/build/head/ExpoHeadModule.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit dd09948

Please sign in to comment.