Skip to content

Conversation

@kimushun1101
Copy link
Member

以前他のissueかpull requestでもお聞きしましたが、miseにRustのバージョン管理も含めてもよいか再度ご検討いただけますでしょうか?
experimentalではありますが、Docker上で動作を確認できました。
https://mise.jdx.dev/lang/rust.html
もし他に悪影響を与えていなければ入れていただけますと幸いです。

確認方法

https://mise.jdx.dev/mise-cookbook/docker.html を参考にDockerでmiseのコンテナーを作成

docker run --pull=always -it --rm --entrypoint bash jdxcode/mise:latest

docker内で以下のコマンドで環境構築

git clone -b feature/add-rust-in-mise https://github.com/typst-jp/typst-jp.github.io.git
cd typst-jp.github.io
mise trust
mise install

ビルドの確認

mise run generate

mise run previewを試すためにはdocker runにポート設定が必要になるかと思いますが、Rustに無関係かと思い試していません。

補足

[tools]に関して、推奨の記述順序あればご指示ください。

Signed-off-by: Shunsuke Kimura <[email protected]>
@kimushun1101 kimushun1101 requested a review from 3w36zj6 May 22, 2025 11:29
@3w36zj6
Copy link
Member

3w36zj6 commented May 22, 2025

以前はmiseはrustupの使用を推奨していましたが、現在はcoreサポートになっている1のでmiseの管理下に追加して良さそうですね。

Footnotes

  1. https://github.com/jdx/mise/pull/3219

.mise.toml Outdated
[tools]
node = "22.11.0"
bun = "1.1.37"
rust = "1.81.0"
Copy link
Member

Choose a reason for hiding this comment

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

v0.13.1のRustのバージョンは1.83なのでこれに揃えるのが無難だと思います。

https://github.com/typst/typst/blob/v0.13.1/.github/workflows/release.yml

.mise.toml Outdated
Comment on lines 1 to 4
[tools]
node = "22.11.0"
bun = "1.1.37"
rust = "1.81.0"
Copy link
Member

Choose a reason for hiding this comment

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

NITS: toolsの順番については特に拘りはないですが、Rustが主となるプロジェクトなのでRustが一番上にあると良いかもしれません。

Signed-off-by: Shunsuke Kimura <[email protected]>
@kimushun1101
Copy link
Member Author

コメントありがとうございます。

  • バージョンを1.83.0
  • 先頭に記載

を行いました。

@3w36zj6
Copy link
Member

3w36zj6 commented May 22, 2025

CI/CDやDev Containerはmiseの設定ファイルを尊重する形で別のPRで修正しましょう。

@3w36zj6 3w36zj6 merged commit 614b2b3 into main May 22, 2025
3 checks passed
@3w36zj6 3w36zj6 deleted the feature/add-rust-in-mise branch May 22, 2025 12:24
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.

3 participants