-
Notifications
You must be signed in to change notification settings - Fork 52
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
更新至 v2.0.1 #245
更新至 v2.0.1 #245
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Worth considering though. View full project report here.
@@ -467,13 +469,13 @@ def _read_user_list(path: Path) -> List[str]: | |||
if not path: | |||
return [] | |||
if os.path.isfile(path): | |||
with open(path, "r", encoding=_conf.preference.encoding) as f: | |||
with open(path, "r", encoding=plugin_config.preference.encoding) as f: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UnicodeDecodeError
can occur if the content of the file has characters incompatible with the OS's default encoding. Python uses the OS's default text encoding on the content because encoding
is not set. Read more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some food for thought. View full project report here.
Closes: 242
[skip ci]
[skip ci]
# Conflicts: # README.md
# Conflicts: # src/nonebot_plugin_mystool/utils/good_image.py
# Conflicts: # README.md # pyproject.toml # src/nonebot_plugin_mystool/_version.py
(cherry picked from commit cd27a63)
No description provided.