-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix/upgrade deps #23
Fix/upgrade deps #23
Conversation
Reviewer's Guide by SourceryThis pull request updates dependencies, refactors the logger, removes Flutter-specific content and example projects, and adjusts workflow configurations. The primary goal is to upgrade the core Updated class diagram for LoggerclassDiagram
class Logger {
+static void log(String message)
+static void error(String message)
+static void warning(String message)
}
note for Logger "Refactored logger class"
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've reviewed this pull request using the Sourcery rules engine. If you would also like our AI-powered code review then let us know.
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
This should be small
Small pull requests are great and easy for me to understand and accept
Please try prefix every commits in the pull request with Arlo's git notation
But it's not small!
Then you should setup a remote pairing session with Llewellyn ( llewellyn.falco@gmail.com )
Usually the sessions are between 45-90 minutes.
assuming you still feel it is small, please include
Description
A description of what the PR achieves.
The solution
Outline the implementation.
Any tests that are affected.
Notation
I prefer lots of very small commits prefixed with Arlo's git notation
Summary by Sourcery
This pull request updates the
ApprovalTests
package to version 1.1.6, upgrades dependencies, improves the README with a link to the new starter project, and enhances the documentation with snippets and examples for usingverifyAsJson
with data models.Enhancements:
ApprovalTests
package to version 1.1.6, including dependency upgrades..gitignore
, linter, and GitHub Actions CI setup.verifyAsJson
with data models, including code and resulting file examples.Build:
sizzle_lints
dependency.working-directory
defaults from the GitHub Actions workflows.