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

Refactor analyze/2 to return a Report struct... #19

Open
kitplummer opened this issue Mar 16, 2020 · 0 comments
Open

Refactor analyze/2 to return a Report struct... #19

kitplummer opened this issue Mar 16, 2020 · 0 comments
Labels
enhancement New feature or request refactor make mo betta stuff kinda worky now

Comments

@kitplummer
Copy link
Collaborator

https://github.com/gtri/lowendinsight/blob/develop/lib/analyzer_module.ex#L13

Need to distinguish between the repo-specific report and the top-level report, which should maybe be Analysis struct.

All the docs will need to be updated, and this is probably something that will need to be deprecated, so adding in as a new function - with deprecation warnings for the current functionality?

@kitplummer kitplummer added the enhancement New feature or request label Mar 16, 2020
@kitplummer kitplummer added the refactor make mo betta stuff kinda worky now label Apr 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request refactor make mo betta stuff kinda worky now
Projects
None yet
Development

No branches or pull requests

1 participant