2DegreesInvesting / tiltPlot

Plots for the TILT project
https://2degreesinvesting.github.io/tiltPlot/
GNU General Public License v3.0
0 stars 0 forks source link

Linda's trial of refactoring the architecture of tilt Plot #36

Closed lindadelacombaz closed 4 months ago

lindadelacombaz commented 10 months ago

This is a draft PR to try to implement some points made from the review of the tiltPlot package made from @maurolepore.

I tried to do the exercise to conceptualize and see how things would work and realized how big of a task it was as a little practice for myself, so thank you @maurolepore !


TODO

EXCEPTIONS

maurolepore commented 9 months ago

@lindadelacombaz

As an example, you may want to see this suggestion by chatGTP, and the blog post The Clean Architecture.

To keep each change small it may be best to do the refactoring in small PRs, each dealing with one part of one function.

lindadelacombaz commented 4 months ago

I will close this PR for now and create proper small, issues for it.