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

タグおよびレシピ回りの拡張を追加 #25

Merged
merged 28 commits into from
Dec 17, 2024
Merged

Conversation

Hiiragi283
Copy link
Contributor

No description provided.

Hiiragi283 and others added 19 commits December 13, 2024 14:44
Experimental: Added new format ingredient
Formatted scripts
Moved mixin classes into tag package
Renamed some fields for RobustumRecipeSerializers.kt
1.16バージョンでは`fabric-loader-junit`がそのままだと正常に動作しない(fabirc-loader#817)ため、先駆者のパッチを適用してfabric-loader自体にASMで書き換えを行った上で動かす必要がある
@Hiiragi283 Hiiragi283 changed the title Add tag/recipe extensions タグおよびレシピ回りの拡張を追加 Dec 15, 2024
@Hiiragi283 Hiiragi283 requested a review from turtton December 15, 2024 05:53
@Hiiragi283
Copy link
Contributor Author

これでどうでしょうか

欠けていたKDocを大体追加
Copy link
Contributor

@turtton turtton left a comment

Choose a reason for hiding this comment

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

全体としてはtestも通ってますし良さそうです

@turtton
Copy link
Contributor

turtton commented Dec 17, 2024

githubにはtestって名前のユーザーがいるせいで@付けたらメンション飛ぶの草。これ罠すぎない? 4c7e7c6

@Hiiragi283
Copy link
Contributor Author

マージはしちゃっていい…のかな?

@turtton
Copy link
Contributor

turtton commented Dec 17, 2024

あ、それなら一旦全部見ますね(Testだけ見てた

@turtton
Copy link
Contributor

turtton commented Dec 17, 2024

これ、今テスト書いてない部分ってワールドロードが必要orバグが起き得るほどの複雑性を持たない自明なコードって認識で大丈夫ですかね?

@Hiiragi283
Copy link
Contributor Author

大体そうですね

Copy link
Contributor

@turtton turtton left a comment

Choose a reason for hiding this comment

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

全体もOKですー
ただ最後に1つ提案だけ。例として一個載せましたがテスト書いてもらった部分全部こんな感じにするのはどうですかね...?

@Hiiragi283
Copy link
Contributor Author

ではマージしますね

@Hiiragi283 Hiiragi283 merged commit ca73c29 into sinoalice Dec 17, 2024
2 checks passed
@Hiiragi283 Hiiragi283 deleted the tag_update branch December 17, 2024 12:33
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