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

Use min-indent #4

Merged
merged 2 commits into from Nov 4, 2017
Merged

Use min-indent #4

merged 2 commits into from Nov 4, 2017

Conversation

jamiebuilds
Copy link
Contributor

@jamiebuilds jamiebuilds commented Oct 27, 2017

I needed to use the first part of this logic for another module I'm working on. So I split it out into a min-indent package. I figured I'd add it to this package as well.

Also @sindresorhus, if you want to have the repo/package I'm happy to transfer it to you. I purposefully copied your setup.

@sindresorhus
Copy link
Owner

@sindresorhus sindresorhus commented Nov 4, 2017

Also @sindresorhus, if you want to have the repo/package I'm happy to transfer it to you. I purposefully copied your setup.

Not needed, but would be cool if you could add me to the repo and npm package though.

@sindresorhus sindresorhus merged commit 089df03 into sindresorhus:master Nov 4, 2017
1 check passed
@sindresorhus
Copy link
Owner

@sindresorhus sindresorhus commented Nov 4, 2017

I needed to use the first part of this logic for another module I'm working on.

Let me know when it's out. I'm curious about what other use-cases there are :)

@jamiebuilds
Copy link
Contributor Author

@jamiebuilds jamiebuilds commented Nov 4, 2017

Added you to min-indent

Also, I'm using it for comment normalization in ASTs

"    /**\n     * test\n     */" <-> "test"

@jamiebuilds jamiebuilds deleted the min-indent branch Nov 4, 2017
@sindresorhus
Copy link
Owner

@sindresorhus sindresorhus commented Nov 4, 2017

Neat

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