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

Fix step kind detection when language is declared in file comment. #40

Merged
merged 2 commits into from
Mar 10, 2021

Conversation

Socolin
Copy link
Collaborator

@Socolin Socolin commented Mar 7, 2021

  • Step kind is now detected during parsing (GherkinParser) and stored inside
    GherkinStep: No more complex code to compute this, it's already computed
  • When parsing language, create a new node GherkinLanguageComment if we
    need to access this later (syntax highlight ?, check language is known ?)
  • Store language into GherkinFile for future use (Keyword completion)

@Socolin Socolin added the enhancement New feature or request label Mar 7, 2021
@Socolin Socolin self-assigned this Mar 7, 2021
@Socolin
Copy link
Collaborator Author

Socolin commented Mar 7, 2021

Fix #27

@Socolin Socolin force-pushed the fix-steps-language branch 2 times, most recently from 83647d4 to c587dbe Compare March 7, 2021 18:48
@Socolin
Copy link
Collaborator Author

Socolin commented Mar 7, 2021

I changed it. It's now using the regex

- Step kind is now detected during parsing (GherkinParser) and stored inside
GherkinStep: No more complex code to compute this, it's already computed
- When parsing language, create a new node GherkinLanguageComment if we
need to access this later (syntax highlight ?, check language is known ?)
- Store language into GherkinFile for future use (Keyword completion)
@nemesv nemesv merged commit 87ddef3 into master Mar 10, 2021
@nemesv nemesv deleted the fix-steps-language branch March 10, 2021 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants