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(modules): Modules create only one cleanup task #2657

Merged
merged 5 commits into from
Dec 1, 2022

Conversation

krystofwoldrich
Copy link
Member

@krystofwoldrich krystofwoldrich commented Dec 1, 2022

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

The cleanup task was missing releaseName and versionCode and was created with the same name for multiple variants.
One cleanup task is enough as there is one source map file.

💡 Motivation and Context

fixes: #2656

💚 How did you test it?

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing
  • No breaking changes

🔮 Next steps

  • Test with an app with multiple variants

CHANGELOG.md Outdated Show resolved Hide resolved
@krystofwoldrich krystofwoldrich marked this pull request as draft December 1, 2022 09:47
@krystofwoldrich krystofwoldrich changed the title fix(modules): Modules cleanup task name includes releaseName and versionCode fix(modules): Modules create only one cleanup task Dec 1, 2022
as that's how it is used in the task names
@krystofwoldrich krystofwoldrich marked this pull request as ready for review December 1, 2022 10:49
@krystofwoldrich krystofwoldrich merged commit 8fbf600 into main Dec 1, 2022
@krystofwoldrich krystofwoldrich deleted the fix-existing-task-modules branch December 1, 2022 12:53
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.

Cannot add task bundleFooBarJsAndAssets_SentryCollectModulesCleanUp
2 participants