search

geerlingguy / sbc-reviews

Jeff Geerling's SBC review data - Raspberry Pi, Radxa, Orange Pi, etc.
MIT License
350 stars 9 forks source link

Patch 1 #31

Closed platima closed 7 months ago

platima commented 7 months ago

Added more details about CPU specs, however, I noted that many of the test outputs you had either had the wrong CPU listed, OR, the early version of the board you had was quite different?

If only Markdown supported sorting tables =/

geerlingguy commented 7 months ago

I think having the CPU / SoC would be helpful—the other information is just a bit too much / overload (and would cause the column width to go wider than the viewport, at least on GitHub).

platima commented 7 months ago

Valid! Could just split it into a new 'Issues' page based from the same CSV that's labelled as 'Detailed View'?

------ Original Message ------ From "Jeff Geerling" @.> To "geerlingguy/sbc-reviews" @.> Cc "Platima" @.>; "Author" @.> Date 26/11/2023 10:59:34 AM Subject Re: [geerlingguy/sbc-reviews] Patch 1 (PR #31)

I think having the CPU / SoC would be helpful—the other information is just a bit too much / overload (and would cause the column width to go wider than the viewport, at least on GitHub).

— Reply to this email directly, view it on GitHub https://github.com/geerlingguy/sbc-reviews/pull/31#issuecomment-1826474555, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADIYAQDWYSNQRPRS6UWRZZLYGKWBNAVCNFSM6AAAAAA72PBW7SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWGQ3TINJVGU. You are receiving this because you authored the thread.Message ID: @.***>

platima commented 7 months ago

Crap, wrong button.

Have simplified, and added link to an 'issue' with full details.