Skip to content

chore: upgrade all dependencies #98

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

chore: upgrade all dependencies #98

wants to merge 3 commits into from

Conversation

@lsaudon
Copy link
Contributor

@lsaudon lsaudon commented Jul 21, 2025

characters and meta is locked by flutter_test

@lsaudon lsaudon force-pushed the chore/dependencies branch from 5085558 to 5f90be7 Compare July 21, 2025 19:52
lsaudon and others added 3 commits July 25, 2025 14:13
@lsaudon
chore: upgrade all dependencies
31fcd93 
characters and meta is locked by flutter_test
@lsaudon
fix(lint): remove exclude
114a922 
Allows you to have the linter in test files that are not generated.
Now, generated code have ignore_for_file: type=lint, type=warning
@lsaudon
chore: migration InMemoryAssetWriter to TestReaderWriter
a75b131
@lsaudon lsaudon force-pushed the chore/dependencies branch from 80524e5 to a75b131 Compare July 25, 2025 12:13
@olexale
Copy link
Owner

olexale commented Aug 19, 2025

Sorry, I just realized I’ve been missing GitHub notifications for over a month. I see the test is failing, and I’ll try to make some time over the weekend to check what’s going on.

Thanks a lot for the PR!

@lsaudon
Copy link
Contributor Author

lsaudon commented Aug 19, 2025

No worries. I think it comes from “migration InMemoryAssetWriter to TestReaderWriter.” I don't know if I did the right things, and I'm reaching my intellectual limits.

@olexale
Copy link
Owner

olexale commented Aug 24, 2025

Apparently, the new build behavior turned out to be more “improved” than I expected, so I ended up making a breaking change by dropping the built-in steps. This allowed me to resolve the dart:mirrors issue that you had in your PR.
I’ve created a new PR, so I’ll close this one.
Thanks a lot for your contribution - it’s much appreciated! And sorry for not noticing your PR earlier.

@olexale olexale closed this Aug 24, 2025
@lsaudon lsaudon deleted the chore/dependencies branch August 24, 2025 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lsaudon @olexale