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

minSdk and targetSdk properties must be supported too #4

Closed
wants to merge 2 commits into from

Conversation

dedece35
Copy link
Member

@dedece35 dedece35 commented Nov 28, 2022

This detection rule triggers on minSdkVersion and targetSdkVersion but their variants minSdk and targetSdk are ignored. This has to be fixed.

initial PR (before migration): cnumr/ecoCode#178
(plop @olegoaer )

This detection rule triggers on `minSdkVersion` and `targetSdkVersion` but their variants `minSdk` and `targetSdk` are ignored. This has to be fixed.
@dedece35 dedece35 added 💉 bug Something isn't working android Something tied to the android OS labels Nov 28, 2022
@jhertout
Copy link
Contributor

jhertout commented Mar 9, 2023

Done in PR #28

@jhertout jhertout closed this Mar 9, 2023
@dedece35
Copy link
Member Author

dedece35 commented Mar 9, 2023

@jhertout @olegoaer @Jberque
now, can I create the release 1.0.0 of android and iOS plugin ?

@jhertout @olegoaer We can delete the branch of this PR. no ? (button "delete branch" bottom)

@jhertout
Copy link
Contributor

For me you can create the 1.0.0. Wait for @olegoaer answer to be sure.

@olegoaer
Copy link
Member

go-ahead 🚀

@jhertout jhertout deleted the olegoaer_main branch March 30, 2023 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android Something tied to the android OS 💉 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants