Skip to content

fix(flutter_plugin): remove static workflows#456

Merged
wolfenrain merged 3 commits intoVeryGoodOpenSource:mainfrom
dewinjm:fix/flutter_plugin_static_workflows
Jul 13, 2022
Merged

fix(flutter_plugin): remove static workflows#456
wolfenrain merged 3 commits intoVeryGoodOpenSource:mainfrom
dewinjm:fix/flutter_plugin_static_workflows

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
Contributor

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
ahsanf pushed a commit to Arkabyte-Teknologi/very_good_cli that referenced this pull request Aug 12, 2025
Co-authored-by: Felix Angelov <felangelov@gmail.com>
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