Skip to content
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: change builder name to flutter_gen_runner (#589) #592

Merged
merged 3 commits into from
Oct 20, 2024

Conversation

b2nkuu
Copy link
Contributor

@b2nkuu b2nkuu commented Oct 19, 2024

What does this change?

Fixes #589 🎯
change builder name to flutter_gen_runner

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • 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 test)
    • Ensure the analyzer and formatter pass (melos run format to automatically apply formatting)
  • Appropriate docs were updated (if necessary)

Copy link

codecov bot commented Oct 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.01%. Comparing base (a5f0e4e) to head (e728872).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #592   +/-   ##
=======================================
  Coverage   98.01%   98.01%           
=======================================
  Files          24       24           
  Lines         905      907    +2     
=======================================
+ Hits          887      889    +2     
  Misses         18       18           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@AlexV525
Copy link
Member

Could we compatible with both names?

@b2nkuu
Copy link
Contributor Author

b2nkuu commented Oct 20, 2024

Could we compatible with both names?

I will try.

Copy link
Member

@AlexV525 AlexV525 left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for the fix

@AlexV525 AlexV525 merged commit 1f4f6cc into FlutterGen:main Oct 20, 2024
5 checks passed
@bartvanandel
Copy link

Thanks for the fix @b2nkuu and for merging it @AlexV525!

One small thing: the test in the (merged) fix contains a typo, "runnfer" should be "runner" in line 233 of asset_gen_test.dart:

        const build = 'test_resources/build_runnfer_assets.yaml';
                                            ~~~~~~~

@AlexV525 AlexV525 mentioned this pull request Oct 20, 2024
1 task
AlexV525 added a commit that referenced this pull request Oct 22, 2024
## What does this change?

- Addresses
#592 (comment)
- Make logs prettier

## Type of change

- [x] Bug fix (non-breaking change which fixes an issue)
@b2nkuu b2nkuu deleted the fix/build_warning branch December 21, 2024 06:13
@AlexV525 AlexV525 added this to the 5.9.0 milestone Feb 5, 2025
@AlexV525 AlexV525 mentioned this pull request Feb 5, 2025
AlexV525 added a commit that referenced this pull request Feb 5, 2025
## 5.9.0

**Feature**
- [#587](#587) Support
Lottie ZIP archive files. by [@AlexV525](https://github.com/AlexV525)
- [#599](#599) Use
`FilterQuality.medium` for the image integration. by
[@AlexV525](https://github.com/AlexV525)
- [#615](#615) Support
`dart_style` v3. by [@AlexV525](https://github.com/AlexV525)
- [#618](#618) Remove
Flare integration. by [@AlexV525](https://github.com/AlexV525)
- [#619](#619) Generate
package path for directory. by [@AlexV525](https://github.com/AlexV525)
- [#620](#620) Format Dart
files with the current Dart version instead of the latest supported. by
[@AlexV525](https://github.com/AlexV525)
- [#621](#621) Adds
`.lottie` support. by [@AlexV525](https://github.com/AlexV525)
- [#635](#635) Support
`archive` v4. by [@AlexV525](https://github.com/AlexV525)
- [#645](#645) Allows not
enable the image integration. by
[@AlexV525](https://github.com/AlexV525)

**Development**
- [#593](#593) Better
stdouts. by [@AlexV525](https://github.com/AlexV525)
- [#622](#622) Fix invalid
codecov config. by [@AlexV525](https://github.com/AlexV525)
- [#630](#630) Update
proper EOF. by [@AlexV525](https://github.com/AlexV525)
- [#643](#643)
Improvements with code lints. by
[@AlexV525](https://github.com/AlexV525)
- [#644](#644) Format
code. by [@AlexV525](https://github.com/AlexV525)

**Bug fix**
- [#592](#592) Accept both
`flutter_gen` and `flutter_gen_runner` as the entry of build.yaml. by
[@b2nkuu](https://github.com/b2nkuu)
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.

[BUG]: Build warning when configuring flutter_gen using build.yaml
3 participants