-
Notifications
You must be signed in to change notification settings - Fork 16
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
Possibility to define barrel file #36
Comments
Would love to see this fixed to allow Sherriff to handle cross-entry-point dependencies without having to rename all my public-api files |
I second this Idea but for different reasons. I am working on a large project with a lot of preexisting index.ts files (that don't respect module boundaries) that prevents me from incrementally applying sheriff. Would be nice to create some barrelIndex.ts that interacts with sheriff while i am cleaning up my codebase. |
@KvikullSigurdur just to avoid a misunderstanding. You can have both barrel modules (with an If you now say that those barrel modules are breaking encapsulation the whole time, then I'd first try to fix that. That said, I think i don't quite catch the idea of the If yes, Let me know if that works for you |
@bradley-carestia-jbh, same goes for you. Just set |
Yep that is the goal :) Jump wanted to know if I could somehow enable the warning incrimentally. barrelFileName works perfectly. Thank you very much! |
Awesome, I'll give that a try next time I'm looking at this. |
index.ts
is the name of the file representing a module.It should be possible to go with another one. For example, the Angular CLI calls it
public-api.ts
.@mikezks: You want to give it a try?
The text was updated successfully, but these errors were encountered: