search

oss-aspen / 8Knot

Dash app in development to serve open source community visualizations using GitHub data from Augur. Hosted app: https://eightknot.osci.io
MIT License
49 stars 60 forks source link

reviewer file heatmap visualization #642

Closed cdolfi closed 7 months ago

JamesKunstle commented 7 months ago

@cdolfi The breakdown into functions that you did actually makes a ton of sense to me, and makes reading the code SO much easier. I made some suggestions about function names and verbally describing the code in the process_data function.

JamesKunstle commented 7 months ago

@cdolfi squash commits as you feel is necessary, rebase onto dev, and this is good to go!

cdolfi commented 7 months ago

@JamesKunstle looking at the commits I think its a solid version history