Rod__
Rod__
Hi, is Rod. Made [this change](https://github.com/rrhg/soorgeon/blob/dev/apply_black_before_refactoring/src/soorgeon/cli.py#L64) & apparently is working, but would like your input. Basically, `clean_mudule.basic_clean(nb_path)` already implements this logic of applying black, so now is also run before...
Fixed unit-tests by adding option to disable string normalizations (aka do not mess with single quotes) when applying black. The reason is that many tests were failing because they use...
Submitted [a PR](https://github.com/ploomber/ci-notebooks/pull/1) to ploomber/ci-notebooks that should fix the integration-tests on Soorgeon. The reason is that those notebooks are downloaded by the Soorgeon integration-tests workflow & converted to dags. Those...
Done. Tests passed!
Thanks. Take your time. Is it posible to assign [issue 65](https://github.com/ploomber/soorgeon/issues/65) to me ?
Would like to take on this challenge. I'm already working on a solution & considering 1st creating a basic implementation with different approaches (parso, plain ast & LibCST) to gain...
Oh, thanks :pray: . Sorry about that, didn't want to waist your time. The thing is, my PR was about no-multi-comp, and it took me a couple of days to...
Need advice on these issues: ## class methods: - Currently, Soorgeon detect globals in functions but not in class methods. - Do we also need to detect & fix globals...