Skip to content

fix: Android 13+ compatibility for ADB Keyboard check#392

Open
JiajunBernoulli wants to merge 1 commit intozai-org:mainfrom
JiajunBernoulli:fix-android-13
Open

fix: Android 13+ compatibility for ADB Keyboard check#392
JiajunBernoulli wants to merge 1 commit intozai-org:mainfrom
JiajunBernoulli:fix-android-13

Conversation

@JiajunBernoulli
Copy link
Copy Markdown

  • Fallback to check package installation when ime list fails due to permission restrictions (SecurityException on Android 13+)

Contribution Guide

We welcome your contributions to this repository. To ensure elegant code style and better code quality, we have prepared
the following contribution guidelines.

What We Accept

  • This PR fixes a typo or improves the documentation (if this is the case, you may skip the other checks).
  • This PR fixes a specific issue — please reference the issue number in the PR description. Make sure your code strictly
    follows the coding standards below.
  • This PR introduces a new feature — please clearly explain the necessity and implementation of the feature. Make sure
    your code strictly follows the coding standards below.

Code Style Guide

Good code style is an art. We have prepared a pre-commit hook to enforce consistent code
formatting across the project. You can clean up your code following the steps below:

pre-commit run --all-files

If your code complies with the standards, you should not see any errors.

Naming Conventions

  • Please use English for naming; do not use Pinyin or other languages. All comments should also be in English.
  • Follow PEP8 naming conventions strictly, and use underscores to separate words. Avoid meaningless names such as
    a, b, c.

For glmv-reward Contributors

Before PR, Please run:

cd glmv-reward/
uv sync
uv run poe lint
uv run poe typecheck

- Fallback to check package installation when ime list fails due to
  permission restrictions (SecurityException on Android 13+)
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.

1 participant