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

Suggestion: Define ESLint library type boundary constraints #46

Open
labeled opened this issue Apr 15, 2021 · 4 comments
Open

Suggestion: Define ESLint library type boundary constraints #46

labeled opened this issue Apr 15, 2021 · 4 comments
Labels
help wanted Extra attention is needed

Comments

@labeled
Copy link

labeled commented Apr 15, 2021

While this does use NX it does not seem to define the boundary constraints between library types. Many ui libraries are depending on data-access libraries and are outside of the scope nx suggests in their documentation that can be found here.
Updating the "depConstraints" to match the suggested dependency constraints will cause eslint to fail.
Dependency Constraints
Feature Library -> can depend on any type of library.
UI Library -> can only depend on other ui or util libraries.
Data-access library -> can only depend on data-access and util libraries.
Utility library -> can only depend on utility libraries.

@trungvose
Copy link
Owner

trungvose commented Apr 16, 2021

@labeled Thank you so much for the suggestion. I am aware of this, and as mentioned on the README, this is my first Nx project, so I want to get something out quickly. That's why I took some shortcuts to reuse the UI component in the same places and plug them directly into the data access.

That said, It would come in a few weeks. If you feel like you want to create a PR in the meantime, feel free. Thanks again :)

@nartc
Copy link
Sponsor Collaborator

nartc commented Jul 3, 2021

The project doesn't have ESLint Boundary setup but most, if not all, libs already have tags defined so applying the boundary should be fairly straight-forward

@trungvose trungvose added the help wanted Extra attention is needed label Jul 7, 2021
@leonlafa
Copy link
Contributor

Please review my PR #77

@trungvose
Copy link
Owner

Thanks @leonlafa, I am looking into it 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants