Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

ChristianKatzmann
Copy link

I have a project with two dimensions. The brands are named in lowerCamelCase, so I may have an uppercase character in the name of the brand.
Unfortunately, the current version of the Play Services Plugin does not work with those names, as we split the variant.dirName by uppercase characters to get all different flavors – which is not correct in my situation.

I refactored the GoogleServicesTask to use the names of the existing fields buildType and productFlavors. Using this approach, we don't have to use regular expressions at all.

@ChristianKatzmann
Copy link
Author

Anything I can do to get this PR merged?

@TacoTheDank
Copy link

@ChristianBecker A lot of waiting apparently 🤔

@ChristianKatzmann
Copy link
Author

@jkasnicki @caller9 @aaronj1335 Friendly ping. This "bug" makes the plugin unusable for me :(

@ChristianKatzmann
Copy link
Author

@caller9 Any chance you can eventually get this PR reviewed and merged? I still can't use the release version of this plugin 👎

@caller9
Copy link
Collaborator

caller9 commented Apr 12, 2021

Another team works on this plugin and they're taking a look. If you could update with the latest changes from master that would help.

@ChristianKatzmann
Copy link
Author

@caller9 Thanks for forwarding this request 👍 I just rebased against latest master.

davidmotson pushed a commit that referenced this pull request Apr 29, 2021
@davidmotson
Copy link
Collaborator

This was merged in #149

@ChristianKatzmann
Copy link
Author

@davidmotson Thank you! Do you plan to release a new version with this fix?

@davidmotson
Copy link
Collaborator

it is now released in 4.3.6

@niparx
Copy link

niparx commented Jul 14, 2021

This was merged in #149

why it is not closed then?

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.

5 participants