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 gherkin lexer #354

Merged
merged 2 commits into from May 1, 2020
Merged

add gherkin lexer #354

merged 2 commits into from May 1, 2020

Conversation

micbar
Copy link
Contributor

@micbar micbar commented Apr 27, 2020

Description

  • Add a gherkin lexer

Related Ticket

Screenshot

Screenshot_2020-04-28

@alecthomas
Copy link
Owner

alecthomas commented Apr 30, 2020

Nice! Mind adding a test?

@micbar
Copy link
Contributor Author

micbar commented Apr 30, 2020

yes. Will add a test.

@micbar
Copy link
Contributor Author

micbar commented May 1, 2020

@alecthomas I added testdata. I saw that most of the lexers are doing it like that.

Is that what you meant?

@alecthomas
Copy link
Owner

alecthomas commented May 1, 2020

Yep perfect, thanks!

@alecthomas alecthomas merged commit 82f38b9 into alecthomas:master May 1, 2020
1 check passed
mrsdizzie pushed a commit to mrsdizzie/chroma that referenced this issue Jul 15, 2020
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.

None yet

2 participants