Skip to content
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

[Fix] Fix griffe version #237

Merged
merged 3 commits into from
Aug 21, 2024
Merged

[Fix] Fix griffe version #237

merged 3 commits into from
Aug 21, 2024

Conversation

fanqiNO1
Copy link
Contributor

@fanqiNO1 fanqiNO1 commented Aug 16, 2024

image

由于 griffe1.0.0 移除了 griffe.enumerations,所以需要修改此处 import。

两种修改方法:

  1. 判断 griffe 版本,如果是 1.x,则直接 import;如果是 0.x,保持原 import 方法。
  2. 直接 import。

本 PR 目前使用了第二种方法。如果佬们觉得第一种方法更合适,可以留下 comment,我来修改为第一种。

@Harold-lkk
Copy link
Member

如果是第二种最好在requirement里边加一个 griffe 版本限制

@fanqiNO1
Copy link
Contributor Author

如果是第二种最好在requirement里边加一个 griffe 版本限制

那要不就直接锁到 >=1.0 版本?

@Harold-lkk
Copy link
Member

如果是第二种最好在requirement里边加一个 griffe 版本限制

那要不就直接锁到 >=1.0 版本?

这个要不然先兼容一下两个版本吧,这个1.0的版本有点新,没发布几天就更新了1.1 感觉不是很稳定

@fanqiNO1
Copy link
Contributor Author

如果是第二种最好在requirement里边加一个 griffe 版本限制

那要不就直接锁到 >=1.0 版本?

这个要不然先兼容一下两个版本吧,这个1.0的版本有点新,没发布几天就更新了1.1 感觉不是很稳定

更新了一版 import 逻辑,辛苦佬再看看

@Harold-lkk Harold-lkk merged commit f01a211 into InternLM:main Aug 21, 2024
1 check passed
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