Skip to content

[Fix] Fix griffe version#237

Merged
Harold-lkk merged 3 commits intoInternLM:mainfrom
fanqiNO1:fix_griffe
Aug 21, 2024
Merged

[Fix] Fix griffe version#237
Harold-lkk merged 3 commits intoInternLM:mainfrom
fanqiNO1:fix_griffe

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
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