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.

Sign up for GitHub

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

Jump to bottom

把Config的type字段默认类型由str改为None #1283

Merged

HibiKier merged 1 commit into HibiKier:main from LambdaYH:main

Feb 28, 2023

Contributor

LambdaYH commented Feb 27, 2023

zhenxun_bot/configs/utils/__init__.py

Lines 186 to 188 in 6826c10

    
           cattrs.structure(config.value, config.type) 
        
           if config.type 
        
           else config.value

看这几行应该是若config指定type才执行转换，反之直接返回值，那么应当就没有必要把Config的默认type全指定为str（会导致第三方老插件不兼容）


          把Config的type字段默认类型由str改为None

33a0477

HibiKier merged commit 401029b into HibiKier:main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet