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

Add showing typo in error message #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adamryman
Copy link
Contributor

@adamryman adamryman commented Jul 14, 2018

I wanted to see what key I hit by mistake.

Up to change implementation if desired, I am liking the result though.

Image of what it looks like in slow mode.

2018-07-14-132313_434x252_scrot

@barnumbirr
Copy link
Contributor

This is a really useful addition. @pb- any plans of merging this?

@adamryman
Copy link
Contributor Author

adamryman commented Aug 17, 2020

@pb- I am so excited to see there have been more pull requests since I worked on this a few years ago!

Do you think we could merge this pull request?

@adamryman
Copy link
Contributor Author

I just rebased on top of master.

I'm not fully sure what to do about how the actual / expected looks.

Input appreciated.
2020-11-02-164216_2560x1440_scrot

@adamryman
Copy link
Contributor Author

@pb- How can we get this merged? What changes would you like to see?

@pb-
Copy link
Owner

pb- commented Nov 24, 2021

@adamryman apologies for responding way too late, you probably already gave up on this. Just in case not - I'm wondering if the mismatch message (expected/actual) is supposed to go away after the penalty delay? It seems that it sticks around, which is a bit confusing.

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