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.

Sign up for GitHub

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

Jump to bottom

Split the file asts in modules before executing the rules #26

Closed
tak30 opened this issue Jul 26, 2022 · 0 comments
Closed

Split the file asts in modules before executing the rules #26

tak30 opened this issue Jul 26, 2022 · 0 comments
Assignees
@pbrudnick
Labels
enhancement New feature or request
Milestone
0.0.1

Comments

@tak30
Copy link
Contributor

tak30 commented Jul 26, 2022

Also we need to take out the modules inside modules
@pbrudnick pbrudnick self-assigned this Jul 26, 2022
@pablocostass pablocostass added the enhancement New feature or request label Jul 26, 2022
pbrudnick added a commit that referenced this issue Jul 26, 2022
@pbrudnick
#26: split file ast by module
d44de91
pbrudnick added a commit that referenced this issue Jul 27, 2022
@pbrudnick @elbrujohalcon
#26: split file ast by module (#28)
edc5f71 
* #26: split file ast by module

* Add spec

* remove test tag

* better naming

* Refactors

* Update lib/meandro/util.ex

Co-authored-by: Brujo Benavides <elbrujohalcon@gmail.com>
Signed-off-by: Pablo Brud <pbrudnick@gmail.com>

* Adding nested modules tests

* Fix dialyzer errors in elixir 1.13 replacing the function

* Addressing a suggestion

* Improving existing tests

* Fixing order

Co-authored-by: Brujo Benavides <elbrujohalcon@gmail.com>
@elbrujohalcon elbrujohalcon added this to the 0.0.1 milestone Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pbrudnick pbrudnick

Labels
enhancement New feature or request
Projects
None yet
Milestone
0.0.1
Development

No branches or pull requests
4 participants
@elbrujohalcon @pbrudnick @tak30 @pablocostass