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 simple head/tail modifiers #2227

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

josharian
Copy link
Collaborator

Checklist

  • I have added tests
  • [-] I have updated the docs and cheatsheet
  • [-] I have not broken the cheatsheet

@josharian
Copy link
Collaborator Author

cc @pokey -- not for review yet, a 📌 for a conversation for a possibly terrible idea i had about handling non-simple head/tail

@josharian josharian added the to discuss Plan to discuss at meet-up label Feb 8, 2024
@pokey
Copy link
Member

pokey commented Apr 22, 2024

I believe we discussed this and decided that it would be better to have one token to indicate the start of a head or tail modifier and one token to indicate the end. If nothing falls in between then it is a head or tail of line, otherwise it's a head or tail of whatever modifier sequence falls in between. Does that line up with your recollection?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keyboard to discuss Plan to discuss at meet-up
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants