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

Optimize code #14

Merged
merged 19 commits into from
Oct 4, 2024
Merged

Optimize code #14

merged 19 commits into from
Oct 4, 2024

Conversation

RawDiamondMC
Copy link
Member

No description provided.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for your first Pull Request! Join our Discord server to get more info.

Copy link
Contributor

@AmarokIce AmarokIce left a comment

Choose a reason for hiding this comment

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

🐺👌

((MapNode) node).forEach(pair -> {
// normalize value
final JsonNode<?> rawValue = pair.getValue();
final List<EntrypointMetadata> entrypointMetadataList = switch (rawValue.getType()) {
Copy link
Contributor

Choose a reason for hiding this comment

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

[待办] 建议分割为小块的私有方法处理,而不是堆叠嵌套。这样的嵌套太复杂了。

Copy link
Member Author

Choose a reason for hiding this comment

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

我不想动这块代码,你帮我改了吧x

Copy link
Member Author

Choose a reason for hiding this comment

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

我自己都快看不懂了

Copy link
Contributor

Choose a reason for hiding this comment

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

打个 TODO 吧,我来解构

Copy link
Member Author

Choose a reason for hiding this comment

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

@AmarokIce AmarokIce self-requested a review September 21, 2024 07:43
Copy link
Contributor

@AmarokIce AmarokIce left a comment

Choose a reason for hiding this comment

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

Keep

@RawDiamondMC
Copy link
Member Author

不是你们怎么都approve了

我还没写完呢

@TexBlock
Copy link
Contributor

不是你们怎么都approve了

我还没写完呢

嘛?

@AmarokIce
Copy link
Contributor

不是你们怎么都approve了

我还没写完呢

你推送一段我们 approve 一段阿(?)

@TexBlock TexBlock self-requested a review October 4, 2024 07:55
@RawDiamondMC
Copy link
Member Author

@AmarokIce 你怎么还没动静 💢

@RawDiamondMC RawDiamondMC merged commit 765092f into dev/1.21 Oct 4, 2024
3 checks passed
@RawDiamondMC RawDiamondMC deleted the feature/hugeChanges branch October 4, 2024 11:08
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.

4 participants