-
Notifications
You must be signed in to change notification settings - Fork 816
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
Update build.gradle #720
base: master
Are you sure you want to change the base?
Update build.gradle #720
Conversation
Sets namespace to fix the following error: ``` Failed to query the value of property 'buildFlowServiceProperty'. > Could not isolate value org.jetbrains.kotlin.gradle.plugin.statistics.BuildFlowService$Parameters_Decorated@325f2fc3 of type BuildFlowService.Parameters > A problem occurred configuring project ':qr_code_scanner'. > Could not create an instance of type com.android.build.api.variant.impl.LibraryVariantBuilderImpl. > Namespace not specified. Specify a namespace in the module's build file. See https://d.android.com/r/tools/upgrade-assistant/set-namespace for information about setting the namespace. If you've specified the package attribute in the source AndroidManifest.xml, you can use the AGP Upgrade Assistant to migrate to the namespace value in the build file. Refer to https://d.android.com/r/tools/upgrade-assistant/agp-upgrade-assistant for general information about using the AGP Upgrade Assistant. ```
@asmrtfm please merge this. |
Merge it! |
Please merge it! |
@asmrtfm please merge this pull request ASAP Its quite urgent Thanks |
@aditya1331 can you able to merge this PR please ? |
Looks like the maintainer is quite stubborn. Maybe they want everyone to migrate to their new package mobile_scanner but that is such a chore. Well thanks @asmrtfm for this patch/PR (saved me a lot of time). If anyone else stumbles onto this post, in your qr_code_scanner: ^1.0.1 with qr_code_scanner:
git:
url: https://github.com/asmrtfm/qr_code_scanner
ref: patch-1
version: ^1.0.0 to use this patch. Maybe also leave a comment thanking @asmrtfm. |
Sets namespace to fix the following error: