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

Recursive project search #85

Merged
merged 2 commits into from
Dec 16, 2020

Conversation

yogsototh
Copy link
Contributor

This PR introduce a /** notation for :project-dirs to recursively search for sub projects.

We have a very big monorepo.
Its size is so big now that we would like to support a hierarchy in those sub projects.
Instead of manually writing all sub directories, I would like to search recursively.

Typically we already have :project-dirs ["lib/*" "services/*"] and we would like to split some services but not have a flat organisation so :project-dirs ["lib/*" "services/**"]. This way we could take a service foo split the code in 10 sub projects under the foo directory (that will no longer be a project directory).

Copy link
Collaborator

@greglook greglook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a nice improvement! It will certainly simplify our metaproject config as well to be able to list just services/**. A few points of feedback to address.

example/libs/subdir/lib-c/src/lib_c/lib_b/core_test.clj Outdated Show resolved Hide resolved
example/project.clj Outdated Show resolved Hide resolved
src/lein_monolith/config.clj Outdated Show resolved Hide resolved
src/lein_monolith/config.clj Outdated Show resolved Hide resolved
Copy link
Collaborator

@greglook greglook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙆‍♂️

@greglook greglook merged commit cf1d40d into amperity:master Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants