-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor(readme): update readme with new information #55
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The project has been inactive for quite a while, updating old commands and dependencies.
Newer linting tools finds new errors
Migrate the version to dart 3.0
Increase version for project.
To match local version used.
The official dart analyzer ruleset still uses the deprecated strong mode rulesets.
Was still using flutter format instead of dart format.
Forgot to update the version for the test pipeline according to the other pipelines.
The cmake paths for the integration tests where corrupted and needed rebuilding.
Old firestore library was deprecated and is now replaced with the recommended approach.
Using the new library. Still needs to be refactored later.
This improves readability of the code.
This should make testing and mocking much cleaner and easier.
This makes debugging much cleaner and allows for further expansions.
This allows for more modularity and flexibility, due to using dependency injection. This should make mocking for tests much cleaner.
This builds upon the dependency injection approach from before.
The functionaility of this class is replaced by the new services.
This allows faster navigation inside of the project.
This approach heavily relies on a self-mocked firebase service, to avoid the use of production databases.
Those errors are not accurate, since the auth config fie is hidden intentionally.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The project has been inactive for quite a while, updating old commands and dependencies.