Skip to content

Conversation

Monther-debug
Copy link

No description provided.

@Monther-debug
Copy link
Author

done

@hoangdat
Copy link
Member

hi @Monther-debug , can you describe your PR?
What bugs you are trying to fix?

@Monther-debug
Copy link
Author

I just fix the packages problem

Copy link
Member

@dab246 dab246 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2025-08-18 at 14 30 40

We don't need this change. It's probably only needed on your project's side.

@Monther-debug
Copy link
Author

ok i will do it

@Monther-debug
Copy link
Author

done

1 similar comment
@Monther-debug
Copy link
Author

done

@chibenwa
Copy link
Member

Thanks a lot for this contribution.

This PR has a title fix the bugs.

Which bugs? Can we have an exhaustive list of bugs fixed? Can we have 1 commit per bugs?

Finally this pull request is not reviewable. The changeset is too huge. I believe this work needs serious rework and especially be split in explicit subparts (pull requests) so that we have a decent chance to review the quality of it and ensure it is safe.

image

=> this is not ok

Thanks for your understanding.

@@ -0,0 +1,28 @@
C/C++ Structured Logr
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am reluctant to commit binaries from external sources.

@@ -39,7 +39,7 @@ dependencies:
### Dependencies from pub.dev ###
cupertino_icons: 1.0.6

flutter_svg: 2.1.0
flutter_svg: ^2.1.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use strict version. Having tolerance here open a vulnerability to potential suply chain attacks.

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