Skip to content

Remove linting step from dangerfile#42332

Closed
motiz88 wants to merge 1 commit into
facebook:mainfrom
motiz88:export-D52834693
Closed

Remove linting step from dangerfile#42332
motiz88 wants to merge 1 commit into
facebook:mainfrom
motiz88:export-D52834693

Conversation

@motiz88
Copy link
Copy Markdown
Contributor

@motiz88 motiz88 commented Jan 17, 2024

Differential Revision: D52834693

Pull Request resolved: #42332

Changelog: [Internal]

TSIA - the Danger ESLint plugin we use is outdated and there have been issues with its reliability. We already have linting jobs on CircleCI that overlap with this functionality.

@facebook-github-bot facebook-github-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 Jan 17, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52834693

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • ⚠️ packages/react-native-bots/dangerfile.js line 106 – 'handleStatuses' is defined but never used. (no-unused-vars)

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52834693

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52834693

@analysis-bot
Copy link
Copy Markdown

analysis-bot commented Jan 17, 2024

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 16,798,992 +0
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 20,190,308 -10
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 6c8dfc8
Branch: main

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52834693

Summary:

Changelog: [Internal]

TSIA - the Danger ESLint plugin we use is outdated and there have been issues with its reliability. We already have [linting jobs on CircleCI](https://github.com/facebook/react-native/blob/6c8dfc89566d27c36f9ca3828bcf3a63dfcb6942/.circleci/configurations/jobs.yml#L5-L74) that overlap with this functionality.

Reviewed By: cortinico

Differential Revision: D52834693
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D52834693

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jan 17, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 8e70acc.

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. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants