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

dataclasses-jsonfireの許容するバージョンの範囲を広げました #139

Merged
merged 2 commits into from
Aug 6, 2024

Conversation

yuji38kwmt
Copy link
Collaborator

@yuji38kwmt yuji38kwmt commented Aug 6, 2024

pyproject.tomldataclasses-json = "^0.5.7"と記載すると、「0.5.7以上、0.6未満」が利用可能なバージョンになります。
annofab-3dpc-editor-cliに依存するPythonツールを開発する際、許容するバージョンの範囲が狭いと使いにくいので(たとえば、dataclasses-jsonの最新バージョンを使いたいのに使えない)ので、許容するバージョンの範囲を広げました。

メジャーバージョンが0の状態ならば、たぶん動作するだろうと考えてdataclasses-json= ">=0.5.7,<1" と記載しました。


補足

動作確認したバージョンのみを許容するのであれば、 dataclasses-json= ">=0.5.7,<1" の方がよいかもしれません。

@yuji38kwmt yuji38kwmt changed the title update dataclass-json 依存ライブラリのdataclasses-jsonfireの許容するバージョンを上げる Aug 6, 2024
@@ -7,7 +7,7 @@


def camelcase(cls: Type[A]) -> Type[A]:
cls.dataclass_json_config = config(letter_case=LetterCase.CAMEL)["dataclasses_json"] # type: ignore[assignment]
cls.dataclass_json_config = config(letter_case=LetterCase.CAMEL)["dataclasses_json"]
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dataclasses-jsonを最新バージョンにしたことで、mypyのエラーが出なくなったのでtype: ignoreコメントを削除しました。

@yuji38kwmt yuji38kwmt marked this pull request as ready for review August 6, 2024 04:11
@yuji38kwmt yuji38kwmt changed the title 依存ライブラリのdataclasses-jsonfireの許容するバージョンを上げる dataclasses-jsonfireの許容するバージョンの範囲を広げました Aug 6, 2024
Copy link
Contributor

@seraphr seraphr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

依存ライブラリのバージョンを上げて、各コマンドが動いてそうなのを確認したぞ

@seraphr seraphr merged commit 8b5631f into master Aug 6, 2024
2 checks passed
@seraphr seraphr deleted the update-dependency branch August 6, 2024 16:36
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