Skip to content

Fix warnings in AdMob plugin and PermissionsUtil#129

Merged
enfipy merged 7 commits intomanifest-permissionsfrom
fix-warning
Aug 3, 2022
Merged

Fix warnings in AdMob plugin and PermissionsUtil#129
enfipy merged 7 commits intomanifest-permissionsfrom
fix-warning

Conversation

@Adoka3710
Copy link
Contributor

@Adoka3710 Adoka3710 commented Aug 1, 2022

Objective

This PR fixes some warnings in the Kotlin code for AdMob Android and PermissionsUtil.

@Adoka3710 Adoka3710 requested a review from enfipy August 1, 2022 11:31
@enfipy enfipy changed the title Fix warning Fix warnings in AdMob plugin and PermissionsUtil Aug 1, 2022
@enfipy enfipy changed the base branch from main to manifest-permissions August 3, 2022 12:27
@enfipy enfipy merged commit 58f9359 into manifest-permissions Aug 3, 2022
@enfipy enfipy deleted the fix-warning branch August 3, 2022 12:27
enfipy added a commit that referenced this pull request Aug 4, 2022
* Bump android manifest version and replace Cargo.toml metadata

* [errors] Alter commands, refactor

* Alter assets folders

* Alter build targets

* Alter docs, add Android and iOS strategies, fix test

* Fix iOS icons bug, alter icons and splash screen, fix iOS Info.plist build, fix other issues

* Fix cargo metadata test

* Refacto android command

* Fix macroquad permissions, refactor errors, remove redundant code

* Fix warnings in AdMob plugin and PermissionsUtil (#129)

* Fix warning

* Fix warning

* Fix warning

* Merg and fix warning

* Add comments what to replace

* Fix warning permissions util

* Add cross-platform permission API in Metadata

* Fix CI build

* Bump versions

* Bump version of apple-bundle

* Small doc fix

* Reorganize tools structure (#132)

* Reorganize tools structure

* Downgrade bundletool version

* Fix import

Co-authored-by: David Ackerman <enfipy@gmail.com>

* Alter description and move rustfmt

* Fix fmt

* Remove log command and add log flag

* Fixes #130, fixes some warnings in AdMob and Crossbow Android

* Closes #102, alter folders in crossbundle-tools, alter features

* Add icon tmp solution

* Remove icons to fix CI

* Update some documentation

Co-authored-by: zupaazmul <78680636+Adoka3710@users.noreply.github.com>
Co-authored-by: Daniil Anikin <81761954+Heezay@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants