Skip to content

Conversation

@renefloor
Copy link
Contributor

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

bug fix

⤵️ What is the current behavior?

Currently pedantic is used for analysis, but that's discontinued

🆕 What is the new behavior (if this is a feature change)?

Now using flutter_lint

💥 Does this PR introduce a breaking change?

no

🐛 Recommendations for testing

📝 Links to relevant issues/docs

🤔 Checklist before submitting

  • All projects build
  • Follows style guide lines (code style guide)
  • Relevant documentation was updated
  • Rebased onto current develop

@codecov
Copy link

codecov bot commented Nov 30, 2021

Codecov Report

Merging #351 (c835a95) into develop (d124fe0) will not change coverage.
The diff coverage is n/a.

❗ Current head c835a95 differs from pull request most recent head 30529df. Consider uploading reports for the commit 30529df to get more accurate results
Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #351   +/-   ##
========================================
  Coverage    75.39%   75.39%           
========================================
  Files           22       22           
  Lines          691      691           
========================================
  Hits           521      521           
  Misses         170      170           
Impacted Files Coverage Δ
...ter_cache_manager/lib/src/compat/file_fetcher.dart 77.77% <ø> (ø)
flutter_cache_manager/lib/src/logger.dart 80.00% <ø> (ø)
...lutter_cache_manager/lib/src/result/file_info.dart 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d124fe0...30529df. Read the comment docs.

@renefloor renefloor enabled auto-merge November 30, 2021 19:08
@renefloor renefloor merged commit aca37f6 into develop Nov 30, 2021
@renefloor renefloor deleted the feature/flutter_lint branch November 30, 2021 19:09
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