-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
minor bug with native messaging example #557
Labels
idle
Issues and pull requests with no activity for three months.
Comments
It looks like this is your first issue. Welcome! 👋 One of the project maintainers will be with you as soon as possible. We appreciate your patience. To safeguard the health of the project, please take a moment to read our code of conduct. |
Thanks for the report. Please submit a PR and I will merge it. |
github-actions
bot
added
the
idle
Issues and pull requests with no activity for three months.
label
May 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
this line will throw when you create required key in
HKEY_LOCAL_MACHINE
instead ofHKEY_CURRENT_USER
( which is allowed ) , so this script should not behave like this :webextensions-examples/native-messaging/check_config_win.py
Line 22 in faadfca
also , as the script is in Python anyway , the whole thing can be simplified using for-else :
let me know if you want me rather create a small PR with this fix
The text was updated successfully, but these errors were encountered: