Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable two lint rules which are being removed. #24

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

srawlins
Copy link
Contributor

Each of these are going to be removed soon.

  • avoid_null_checks_in_equality_operators
  • unsafe_html

Work towards dart-lang/sdk#59514 and dart-lang/sdk#57099

Each of these are going to be removed soon.

* avoid_null_checks_in_equality_operators
* unsafe_html

Work towards https://github.com/dart-lang/linter/issues/5063 and https://github.com/dart-lang/linter/issues/5001
@srawlins srawlins mentioned this pull request Nov 15, 2024
15 tasks
@srawlins
Copy link
Contributor Author

srawlins commented Nov 1, 2024

@Goddchen please review. This PR is blocking the removal of these lint rules in the Dart SDK, as flutter-rfb is in Flutter's customer test registry.

auto-submit bot pushed a commit to flutter/tests that referenced this pull request Nov 7, 2024
Work towards https://github.com/dart-lang/linter/issues/5063 and https://github.com/dart-lang/linter/issues/5001

There are open PRs for blocked Dart SDK work in these two repos:

* Goddchen/dart-rfb#26
* Goddchen/flutter-rfb#24

@Goddchen has not responded to these in 72 hours. This change disables the tests. The tests can be re-enabled when those PRs have been merged, and the subsequent relevant commits in this repo bumped.
Copy link
Owner

@Goddchen Goddchen left a comment

Choose a reason for hiding this comment

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

Thanks for creating this PR!

@Goddchen Goddchen merged commit de2e39d into Goddchen:main Nov 13, 2024
6 checks passed
auto-submit bot pushed a commit to flutter/tests that referenced this pull request Nov 19, 2024
Since I have merged this PR that removes the obsolete lint rules: Goddchen/flutter-rfb#24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants