Skip to content

feat: Add matchIndentation option #31

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

Closed
wants to merge 1 commit into from

Conversation

pgoforth
Copy link

@pgoforth pgoforth commented Jun 8, 2023

Closes: #9

I see a few PRs for this issue, and all closed without being merged. I do not know the reasoning:
#11
#25
#28

This PR makes the minimal required changes, includes logic from https://github.com/sindresorhus/indent-string, and adds unit tests, bringing coverage to almost complete (2 branches uncovered, and one is from a typedef).

Screenshot 2023-06-08 at 12 07 29 PM

@pgoforth
Copy link
Author

pgoforth commented Jun 8, 2023

In testing the build against actual code, validation seems to fail after the format. I'm verifying my changes, and will update with results.

Edit: All seems to be working on real code now

@pgoforth
Copy link
Author

pgoforth commented Aug 2, 2023

@gajus The other merged commit does not close this PR. Is there something wrong with your automation?

Edit: I will test the new version with my existing repo and verify the base indent fixing works.

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.

Option for format rule for matching indentation?
1 participant