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

chore(deps): update dev dependency flutter_lints to 5.0.0, solve warnings #2292

Merged
merged 3 commits into from
Sep 27, 2024

Conversation

EchoEllet
Copy link
Collaborator

@EchoEllet EchoEllet commented Sep 27, 2024

Description

Update lints and flutter_lints. to version 5.0.0

  • Add new rule avoid_web_libraries_in_flutter to prevent using dart built-in web libraries in non-web flutter plugin which might cause build failure for non-web platforms if conditional import is not used. This warning will only occur for web imports from Dart and doesn't support web. Building the example app in CI is still needed.
  • Remove redundant error block ignore since it doesn't produce any errors or warnings anymore, seems outdated
  • Use the latest version of flutter_lints (development dependency) for all packages except dart_quill_delta.

This is not a breaking change and is used only during development.

  • New feature: Adds new functionality without breaking existing features.
  • 🛠️ Bug fix: Resolves an issue without altering current behavior.
  • 🧹 Code refactor: Code restructuring that does not affect behavior.
  • Breaking change: Alters existing functionality and requires updates.
  • 🧪 Tests: Adds new tests or modifies existing tests.
  • 📝 Documentation: Updates or additions to documentation.
  • 🗑️ Chore: Routine tasks, or maintenance.
  • Build configuration change: Changes to build or deploy processes.

@EchoEllet EchoEllet marked this pull request as ready for review September 27, 2024 14:57
@EchoEllet EchoEllet merged commit 5d7aee3 into master Sep 27, 2024
2 checks passed
@EchoEllet EchoEllet deleted the chore/update-lints branch September 27, 2024 14:58
CatHood0 pushed a commit to CatHood0/flutter-quill that referenced this pull request Oct 28, 2024
…ings (singerdmx#2292)

* chore(deps): update dev dependency flutter_lints to 5.0.0, solve warnings

* chore: remove redundant analysis error ignore

* chore: add avoid_web_libraries_in_flutter rule, remove empty analyzer declaration
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.

1 participant