Skip to content

Conversation

@behowell
Copy link
Contributor

Previous Behavior

Builds are failing on windows machines, with an error like:

verb @fluentui/react-label build |  [11:12:26 AM] x Error detected while running 'babel:postprocess'
verb @fluentui/react-label build |  [11:12:26 AM] x ------------------------------------
verb @fluentui/react-label build |  [11:12:26 AM] x node:internal/modules/cjs/loader:959
verb @fluentui/react-label build |    throw err;
verb @fluentui/react-label build |    ^
verb @fluentui/react-label build |
verb @fluentui/react-label build |  Error: Cannot find module '..\..\../packages/react-components/react-theme/lib/index.js'

New Behavior

Modify the change from #27313 to use path.resolve to ensure the slashes are in the correct direction on Windows.

Related Issue(s)

@behowell behowell marked this pull request as ready for review March 31, 2023 19:02
@behowell behowell requested a review from a team as a code owner March 31, 2023 19:02
@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 77e7f8b:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@size-auditor
Copy link

size-auditor bot commented Mar 31, 2023

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: f972b0d64f37f8e1812e118ba0d69088901a2700 (build)

@fabricteam
Copy link
Collaborator

📊 Bundle size report

🤖 This report was generated against f972b0d64f37f8e1812e118ba0d69088901a2700

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-northstar)

⚠️ No perf measurements available

@behowell behowell merged commit 34d6188 into microsoft:master Mar 31, 2023
@behowell behowell deleted the build/fix-babel-windows-error branch March 31, 2023 20:07
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Apr 4, 2023
* master: (39 commits)
  Added merge tests for custom styles (microsoft#27086)
  fix: make Option's checkIcon slot render conditionally (microsoft#27409)
  docs: add subcomponents to Combobox and Dropdown docs (microsoft#27410)
  Breadcrumb spec (microsoft#26742)
  bugfix(react-utilities): increments SlotRenderFunction signature to include children (microsoft#27377)
  fix: Menu should not steal focus on close (microsoft#27414)
  Fixed Overflow component when there's more then one children element (microsoft#27402)
  feat(scripts-executors): support custom base and printing all affected packages for checkIfPackagesAffected (microsoft#27364)
  applying package updates
  feat(scripts-tasks): make perf test setup generic (microsoft#26920)
  fix: ColorPicker preview color updated to ignore forced colors (microsoft#27394)
  fix: Fix babel build error on windows (microsoft#27406)
  chore(react-datepicker-compat): Add vr-tests and bundle-size fixtures for DatePicker (microsoft#27351)
  fix(react-dialog): DialogContent scrollbar always visible (microsoft#27367)
  docs: Add tokens search (microsoft#27265)
  chore(scripts-tasks): remove not needed api-extractor task logs (microsoft#27094)
  chore(react-tree): adds e2e flat tree tests (microsoft#27318)
  applying package updates
  Accessibility fix for empty vertical bar chart (microsoft#26254)
  fix(react-datepicker-compat): Fix theming issues with all themes and WHCM (microsoft#27361)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants