Skip to content

Comments

Type hints Fix#1698

Merged
sijis merged 3 commits intoerrbotio:masterfrom
ZhipengXue97:master
May 21, 2025
Merged

Type hints Fix#1698
sijis merged 3 commits intoerrbotio:masterfrom
ZhipengXue97:master

Conversation

@ZhipengXue97
Copy link
Contributor

@ZhipengXue97 ZhipengXue97 commented Jul 10, 2024

Description

Fix type check warning reported by Pyre@Google, which was outdated after code modification.

Detail

update the return type of function _read_dict() from dict to Mapping, since the condition check of the return new_dict and type Mapping after commit 9193b1f

@sijis sijis merged commit 47603b8 into errbotio:master May 21, 2025
5 checks passed
@sijis
Copy link
Contributor

sijis commented May 21, 2025

@ZhipengXue97 thank you!

kayman-mk pushed a commit to Hapag-Lloyd/errbot that referenced this pull request Jul 2, 2025
* Type hints Fix

* style: sort/ruff files

* docs: add info to CHANGES

---------

Co-authored-by: Sijis Aviles <sijis.aviles@gmail.com>
(cherry picked from commit 47603b8)
kayman-mk pushed a commit to Hapag-Lloyd/errbot that referenced this pull request Jul 2, 2025
* Type hints Fix

* style: sort/ruff files

* docs: add info to CHANGES

---------

Co-authored-by: Sijis Aviles <sijis.aviles@gmail.com>
(cherry picked from commit 47603b8)
kayman-mk pushed a commit to Hapag-Lloyd/errbot that referenced this pull request Jul 2, 2025
* Type hints Fix

* style: sort/ruff files

* docs: add info to CHANGES

---------

Co-authored-by: Sijis Aviles <sijis.aviles@gmail.com>
(cherry picked from commit 47603b8)
kayman-mk pushed a commit to Hapag-Lloyd/errbot that referenced this pull request Jul 2, 2025
* Type hints Fix

* style: sort/ruff files

* docs: add info to CHANGES

---------

Co-authored-by: Sijis Aviles <sijis.aviles@gmail.com>
(cherry picked from commit 47603b8)
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.

2 participants