jolav / codetabs

Free Online Services. Github/GitLab star history. Count Lines of Code. CORS proxy server. IP GeoLocation. HTTP Headers. Random Data. Api weather temp. Alexa ranking.
https://codetabs.com
BSD 3-Clause "New" or "Revised" License
236 stars 30 forks source link

Suggestion: default the pie chart to non-blank lines (that is including comments), and provide a toggle #10

Closed ell1e closed 3 years ago

ell1e commented 4 years ago

I suggest that the pie chart should default to showing to non-blank lines including comments, rather than as currently only the actual lines containing code. I think it's the most useful metric, since comments aren't easier to write than code, and a lack of comments also suggests less fleshed out code IMHO. It might be useful to also provide a toggle for those who prefer other metrics like the old one.

jolav commented 4 years ago

I like the idea. Since I have to replace counting tool (is inactive) i willl implement it