Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
33980e6 to
21faacf
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #345 +/- ##
=======================================
Coverage 91.19% 91.19%
=======================================
Files 98 98
Lines 3849 3849
Branches 1964 1964
=======================================
Hits 3510 3510
Misses 339 339
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
marcelomendoncasoares
left a comment
There was a problem hiding this comment.
Great UX with the snip tool!! Next step for the future is to have it in a pre-commit hook so we ensure to never have conflicts on CI. I have only one question, though, that I think would be nice to preserve on the snippets as they were.
| run: >- | ||
| dart format --output=none --set-exit-if-changed . && | ||
| dart pub global activate --source git https://github.com/nielsenko/snip.git && | ||
| snip format . --no-apply |
There was a problem hiding this comment.
Question: Is it possible to preserver trailing commas on the formatting? I would say that some of the snippets lost a bit in readability due to not preserving them.
Uh oh!
There was an error while loading. Please reload this page.