Skip to content

Commit 1f6940e

Browse files
tarunrajputszymonrybczak
authored andcommitted
address feedbacks
1 parent ba99a67 commit 1f6940e

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

packages/cli-doctor/src/tools/healthchecks/__tests__/dependencies.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ describe('dependencies', () => {
3636
expect(diagnostics.needsToBeFixed).toBe(true);
3737
});
3838

39-
it('returns true if dependencies contains an compatible version react native package', async () => {
39+
it('warn if dependencies contains an compatible version of react native packages', async () => {
4040
dependenciesJSON = JSON.stringify({
4141
name: 'AwesomeProject',
4242
dependencies: {
@@ -46,6 +46,8 @@ describe('dependencies', () => {
4646
});
4747

4848
const diagnostics = await dependencies.getDiagnostics(environmentInfo);
49-
expect(diagnostics.needsToBeFixed).toBe(true);
49+
expect(diagnostics.description).toMatch(
50+
'@react-native/codegen is part of React Native and should not be a dependency in your package.json',
51+
);
5052
});
5153
});

packages/cli-doctor/src/tools/healthchecks/dependencies.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ const cliPackages = [
2727
'@react-native-community/cli-tools',
2828
'@react-native-community/cli-doctor',
2929
'@react-native-community/cli-hermes',
30-
'@react-native-community/cli-plugin-metro',
3130
'@react-native-community/cli-clean',
3231
'@react-native-community/cli-config',
3332
'@react-native-community/cli-debugger-ui',

0 commit comments

Comments
 (0)