Skip to content

Commit

Permalink
Revert cra-template workaround (#21550)
Browse files Browse the repository at this point in the history
  • Loading branch information
ecraig12345 committed Feb 3, 2022
1 parent de4f240 commit c5059e1
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 11 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "Revert adding resolution for babel because bug is fixed",
"packageName": "@fluentui/cra-template",
"email": "elcraig@microsoft.com",
"dependentChangeType": "patch"
}
3 changes: 0 additions & 3 deletions packages/cra-template/template.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,6 @@
"typescript": "^4.1.2",
"web-vitals": "^1.0.1"
},
"resolutions": {
"@babel/core": "7.16.12"
},
"eslintConfig": {
"extends": ["react-app", "react-app/jest"]
}
Expand Down
8 changes: 0 additions & 8 deletions packages/fluentui/projects-test/src/createReactApp.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,14 +30,6 @@ export async function createReactApp() {

await prepareCreateReactApp(tempPaths, 'typescript');
const testAppPath = config.paths.withRootAt(tempPaths.testApp);

// TODO: remove once babel issue is fixed (tracked by https://github.com/microsoft/fluentui/issues/21546)
logger('Add resolution to work around @babel/core issue');
const packageJson = fs.readJSONSync(testAppPath('package.json'));
packageJson.resolutions = { '@babel/core': '7.16.12' };
fs.writeJSONSync(testAppPath('package.json'), packageJson, { spaces: 2 });
await shEcho('yarn', testAppPath());

logger(`Test React project is successfully created: ${testAppPath()}`);

logger('STEP 2. Add Fluent UI dependency to test project..');
Expand Down

0 comments on commit c5059e1

Please sign in to comment.