Skip to content

Conversation

@giboin
Copy link
Contributor

@giboin giboin commented Oct 15, 2025

Description

The dart lints were gone 😢

- avoid_implementing_value_types:
        excludes:
          - '**m/*mocks.dart'

was breaking the yaml file, and the linter section was not listened anymore

Instead, let's ignore the lint in mock files of each project

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

@MaximeRougieux
Copy link

Can you add the Changelog also (as you have bumped the version in this PR)?

@giboin giboin changed the title Fix/re add dart lints fix: re add dart lints Oct 15, 2025
@giboin giboin force-pushed the fix/re-add-dart-lints branch from 4fc0de4 to 7837751 Compare October 15, 2025 15:44
@giboin giboin force-pushed the fix/re-add-dart-lints branch from 7837751 to e5d07cf Compare October 16, 2025 12:17
@giboin giboin force-pushed the fix/re-add-dart-lints branch from e5d07cf to 5047a17 Compare October 16, 2025 12:19
@giboin giboin merged commit 668bcce into main Oct 16, 2025
3 checks passed
@giboin giboin deleted the fix/re-add-dart-lints branch October 16, 2025 12:34
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.

4 participants