-
Notifications
You must be signed in to change notification settings - Fork 64
add an option for disable shift + clear #368
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
add an option for disable shift + clear #368
Conversation
|
Yaya-Cout
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is some things to change and I didn't built, but it looks OK.
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
|
|
|
i hope i havent break all |
|
cool is ok |
|
|
Yaya-Cout
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, why is it named "clearalphalockshift" ?
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
|
ok is beacaus i have change function name and e not variable name |
|
1 similar comment
|
|
|
thank you for your patience I think everything is ok now |
|
|
why the test dosent complet ? |
Co-authored-by: Quentin <github@arra.red>
|
thx |
Yaya-Cout
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please apply given suggestions before marking reviews as resolved and requesting a new review
Co-authored-by: Quentin <git@quentinguidee.dev>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
|
||.text|.rodata|.bss|.data|Total (RAM)|Total (ROM)|%0A|-|-:|-:|-:|-:|-:|-:|%0A|Base|822476 bytes|478317 bytes|226520 bytes|1820 bytes|228340 bytes|1302613 bytes|%0A|Head|822644 bytes|478605 bytes|226520 bytes|1820 bytes|228340 bytes|1303069 bytes|%0A||+168 bytes|+288 bytes|+0 bytes|+0 bytes|+0 bytes|+456 bytes|%0A||+0.0 %25|+0.1 %25|+0.0 %25|+0.0 %25|+0.0 %25|+0.0 %25|%0A |
|
I think i have fix all. sorry for the last time. |
Yaya-Cout
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could merge the code right now, just a few i18n edits to stay consistent.
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
Co-authored-by: Yaya-Cout <67095734+Yaya-Cout@users.noreply.github.com>
|
i have fix the name |
Yaya-Cout
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I just forgot about the English translation during my last review. I've tested on the simulator, everything seems fine and ready to merge
i have code this issue #285