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

Draft: Improving indent file to more indent scenarios if there is a rule line above #51

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

Conversation

nbossard
Copy link
Contributor

No description provided.

@nbossard
Copy link
Contributor Author

This MR is to indent rules lines but not only... according to https://cucumber.io/docs/gherkin/reference/ when there is a rule line, "scenario" lines below should be indented more. This is the what is introduced here.

tpope added a commit that referenced this pull request Mar 25, 2023
tpope added a commit that referenced this pull request Mar 25, 2023
@tpope
Copy link
Owner

tpope commented Mar 25, 2023

This approach will only work for English, and I worry that extending it to use syntax matching for other languages won't work correctly when the Rule: heading is far offscreen and thus not highlighted.

I've made my own attempt in the indent-rules branch. It turned into a pretty substantial refactor, so it needs vetting and testing before I merge.

@nbossard
Copy link
Contributor Author

Ok great, I will look at your changes and test it.

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