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(flutter_plugin): remove static workflows #456

Merged
merged 3 commits into from
Jul 13, 2022
Merged

fix(flutter_plugin): remove static workflows #456

merged 3 commits into from
Jul 13, 2022

Conversation

dewinjm
Copy link
Contributor

@dewinjm dewinjm commented Jul 7, 2022

Description

Remove static workflows paths my_plugin.yaml, my_plugin_android.yaml, ...

Because when run for example: very_good create abc_plugin -t flutter_plugin keep my_plugin... files even though the plugin name is abc_plugin.

For further details, see my comment here: #424 (comment)

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

@dewinjm dewinjm requested a review from felangel as a code owner July 7, 2022 22:58
@wolfenrain
Copy link
Member

Thank you for your contribution!

This change should be applied to the generator script on the very_good_flutter_plugin so that we can then update the bundles based on that. I already added changed this for the fix for #424 (VeryGoodOpenSource/very_good_flutter_plugin#41) but to ensure that we don't lose your contribution we can merge this if you like. Can you resolve the conflicts on your branch?

Once again thanks for this PR!

# Conflicts:
#lib/src/commands/create/templates/flutter_plugin/flutter_plugin_bundle.dart
@wolfenrain wolfenrain merged commit fc41f17 into VeryGoodOpenSource:main Jul 13, 2022
@felangel felangel mentioned this pull request Jul 19, 2022
7 tasks
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