Skip to content

feat: support flutter_svg 2.0.0 #350

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

Merged
merged 6 commits into from
Feb 4, 2023
Merged

feat: support flutter_svg 2.0.0 #350

merged 6 commits into from
Feb 4, 2023

Conversation

lcdsmao
Copy link
Member

@lcdsmao lcdsmao commented Jan 24, 2023

What does this change?

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes #<issue_number_goes_here> 🎯

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
    • Ensure the tests (melos run unit:test)
    • Ensure the analyzer and formatter pass (melos run format to automatically apply formatting)
  • Appropriate docs were updated (if necessary)

@lcdsmao lcdsmao marked this pull request as ready for review February 4, 2023 03:52
@lcdsmao lcdsmao requested a review from wasabeef February 4, 2023 03:52
@codecov
Copy link

codecov bot commented Feb 4, 2023

Codecov Report

Merging #350 (b8e4735) into main (a8d847d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #350   +/-   ##
=======================================
  Coverage   98.26%   98.26%           
=======================================
  Files          20       20           
  Lines         693      693           
=======================================
  Hits          681      681           
  Misses         12       12           
Impacted Files Coverage Δ
packages/core/lib/generators/generator_helper.dart 100.00% <ø> (ø)
...e/lib/generators/integrations/svg_integration.dart 100.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@wasabeef
Copy link
Member

wasabeef commented Feb 4, 2023

@lcdsmao Thank you.

@jayjah
Copy link

jayjah commented Feb 4, 2023

Do you have any timeline to merge and publish (to pub.dev) this PR?

@lcdsmao lcdsmao merged commit 98d8617 into main Feb 4, 2023
@lcdsmao lcdsmao deleted the flutter-svg-2.0 branch February 4, 2023 13:41
@lcdsmao
Copy link
Member Author

lcdsmao commented Feb 4, 2023

@wasabeef

Sorry, I merged via Github App accidentally.
Please review it.🥲

@wasabeef wasabeef added this to the 5.2.0 milestone Feb 7, 2023
@wasabeef wasabeef mentioned this pull request Feb 7, 2023
8 tasks
@suxoikorm
Copy link

suxoikorm commented Feb 10, 2023

@wasabeef Hello. This PR led to the fact that my generation broke. It generates a colorFilter no matter what version of flutter_svg I have. Such changes should be made in major versions. Now I have to update flutter_svg in all my packages.

@wasabeef
Copy link
Member

wasabeef commented Feb 11, 2023

@suxoikorm

Sorry for the lack of explanation.
I should have put it in the CHANGELOG.md.

ankiimation pushed a commit to ankiimation/flutter_gen that referenced this pull request Dec 1, 2023
* feat: support flutter_svg 2.0.0

* feat: add lint ignore

* chore: update example pubspec

* chore: build runner exec result

* test: update test resource with svg 2.0.0
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