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

✨ Make barrel header optional #376

Merged
merged 4 commits into from
Oct 8, 2022
Merged

✨ Make barrel header optional #376

merged 4 commits into from
Oct 8, 2022

Conversation

thekarel
Copy link
Contributor

@thekarel thekarel commented Oct 3, 2022

First of all, thanks for the great tool!

Motivation for this PR: in our project there is no real need to know where a barrel file came from. We have existing barrel files without such comments and the fact that some files do have special comments feels odd and will raise eyebrows. Also, in the future we might use different tools to create these.

The PR adds a new CLI flag noHeader that can be used to suppress the comments on top of index.ts files.

@BitForger BitForger self-assigned this Oct 7, 2022
@BitForger BitForger added the Feature Adds a new feature label Oct 7, 2022
@thekarel thekarel requested review from BitForger and removed request for bencoveney October 8, 2022 06:47
@BitForger BitForger changed the title Make barrel header optional ✨ Make barrel header optional Oct 8, 2022
@BitForger BitForger merged commit b8e8b43 into bencoveney:master Oct 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants