Skip to content

Conversation

@FlowingSPDG
Copy link

こんばんは!
volcago のコードリーディングをするうちにここに辿り着いた者です、いつも素晴らしいOSSの開発をありがとうございます!

突然となりますが、本packageのGetGoModule 関数が go.mod へのtoolchain キーや、マイナーバージョンの追加による桁数変更(go 1.14 からgo 1.22.3など)の影響を受け、Go 1.22.3 において使用が出来なくなっていたため、勝手ながら最新化を行うPRを作成させていただきました。

変更内容としては、golang.org/x/mod の最新化、ioutil パッケージの廃止となります。
また、テストケースも追加させていただきましたので合わせてご確認いただけますと幸いです。

お忙しい中突然失礼いたしました、よろしくお願いいたします!

@FlowingSPDG
Copy link
Author

FlowingSPDG commented Jun 12, 2024

また(ご迷惑でなければ)こちらのPRのマージ後に https://github.com/go-generalize/go-easyparser が使用するgo-utils/gopackages もマージ後のバージョンを使用したいと思っており...
趣味で開発しているプロジェクトでvolcago を採用しているのですが、Go1.22.3現在のgo.mod ファイルだと機能せず、gopackages -> go-easyparser -> volcago のような順序でそれぞれGo1.22.3対応を実施したいという意図となります。

また、宜しければeasyparser,volcago 側の最新化対応も私の方でPRの作成などを実施しようかと思っております。
お忙しい中すみません、よろしくお願いいたします。

@tsuzu
Copy link

tsuzu commented Jun 12, 2024

PRありがとうございます!
CIがこけてしまったのでお手数ですが fix-ci-pipeline のbranchを取り込んでもらえますか?

@FlowingSPDG
Copy link
Author

@tsuzu こちらからお声掛けしたのにお待たせしてしまい申し訳ありません...! 🙇
ブランチ取り込みの方実施いたしました!

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