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

同じ優先度のとき、別名スタンプは優先度を低めに並べるように #4136

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

mehm8128
Copy link
Member

@mehm8128 mehm8128 commented Nov 1, 2023

close #3992
優先度0同士で比較するときは多分これでいいのですが、それ以外はあんまり自信ないです

image

Copy link

github-actions bot commented Nov 1, 2023

Copy link

codecov bot commented Nov 1, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!.

Copy link
Member

@ras0q ras0q left a comment

Choose a reason for hiding this comment

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

動作確認しました
完全一致前方一致はこの変更の方が嬉しそうです
それ以外はどっちでもいいけどまあこれで良さそう

@mehm8128 mehm8128 merged commit 49c1826 into master Nov 8, 2023
12 checks passed
@mehm8128 mehm8128 deleted the fix/stamp_list_priority branch November 8, 2023 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants