Skip to content

Exclude generated code from lint#54530

Closed
Simek wants to merge 1 commit into
mainfrom
@simek/lint-ignore-generated-code
Closed

Exclude generated code from lint#54530
Simek wants to merge 1 commit into
mainfrom
@simek/lint-ignore-generated-code

Conversation

@Simek

@Simek Simek commented Nov 13, 2025

Copy link
Copy Markdown
Collaborator

Summary:

I was experimenting with TS API in the monorepo, and spotted that root lint tries to format generated code.

This small change add exclusion for files in private/react-native-codegen-typescript-test/lib which are generated locally when running TS API tests. The path is already ignored in .gitignore file:

Changelog:

[INTERNAL] [FIXED] - Exclude generated code from lint checks to avoid unnecessary warnings.

Test Plan:

Running yarn lint at monorepo root no longer outputs warnings.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 13, 2025
@facebook-github-bot facebook-github-bot added p: Expo Partner: Expo Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Nov 13, 2025

@cipolleschi cipolleschi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix

@meta-codesync

meta-codesync Bot commented Nov 13, 2025

Copy link
Copy Markdown

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this in D86963479.

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review automatically exported from Phabricator review in Meta.

@meta-codesync meta-codesync Bot closed this in 810f983 Nov 13, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Nov 13, 2025
@meta-codesync

meta-codesync Bot commented Nov 13, 2025

Copy link
Copy Markdown

@cipolleschi merged this pull request in 810f983.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @Simek in 810f983

When will my fix make it into a release? | How to file a pick request?

douglowder pushed a commit to react-native-tvos/react-native-tvos that referenced this pull request Nov 15, 2025
Summary:
I was experimenting with TS API in the monorepo, and spotted that root lint tries to format generated code.

This small change add exclusion for files in `private/react-native-codegen-typescript-test/lib` which are generated locally when running TS API tests. The path is already ignored in `.gitignore` file:
* https://github.com/facebook/react-native/blob/main/.gitignore#L146-L146

## Changelog:

[INTERNAL] [FIXED] - Exclude generated code from lint checks to avoid unnecessary warnings.

Pull Request resolved: react/react-native#54530

Test Plan: Running `yarn lint` at monorepo root no longer outputs warnings.

Reviewed By: cortinico

Differential Revision: D86963479

Pulled By: cipolleschi

fbshipit-source-id: e067bbbbf69d24929095aed1145f41ae9f26fbd4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. p: Expo Partner: Expo Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants