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

chore/test: add line of emojis to show multiwidth characters causing … #144

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

Conversation

drahnr
Copy link
Owner

@drahnr drahnr commented Jan 22, 2021

…offsets

What does this PR accomplish?

  • 📙 Documentation
  • 🪣 Misc

Changes proposed by this PR:

Showcases issues as described #141 and #145

📜 Checklist

  • Works on the ./demo sub directory
  • Test coverage is excellent and passes
  • Documentation is thorough

@drahnr drahnr added the bug Something isn't working label Jan 22, 2021
@drahnr drahnr requested review from KuabeM and laysauchoa January 22, 2021 14:37
@KuabeM KuabeM self-requested a review January 22, 2021 17:42
@drahnr
Copy link
Owner Author

drahnr commented Jan 22, 2021

While the test is ok, we should keep it out of master until we have solutions for the referenced issues, so releasing more patches remains possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants