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

Discover module #3

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

ProbablyClem
Copy link
Collaborator

Depends on this pr

Big PR that aim at being able to parse folders directly.
Currently you parse modules but using module.content only works if you have a module inside the file.

Now you can specify a directory and it will parse every files recursively.

With this new implementation, I don't need the module specification anymore because I infer it from the path, so I simplified the attributes with a new paths = "./paths1, ./path2" format which is easier to use.

The old syntax still works, I just ignore the module part.

For easier debugging, I splitted the project in 3 crates, which allows me to have most of the logic in a non-macro crate, where I can use print!(). It's way easier to debug that way !

I bumpted that projet to 4.0 because it's a big change

@ProbablyClem ProbablyClem mentioned this pull request Nov 29, 2023
@rxdiscovery
Copy link
Owner

Hello,

have you thought about external dependencies? that are not on the same crate?

@ProbablyClem
Copy link
Collaborator Author

What do you mean by that ?
You mean subcrates ? Or external crates ?

@rxdiscovery
Copy link
Owner

for example, if you divide your project into several separate sub-projects, with the DTOs in one library and the controllers in another, how will you manage this?

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.

2 participants