CSV rendering bug in GitHub Web App #22405
Replies: 9 comments 11 replies
-
SteveALee:
This is the only solution. The reason why only that line is showing error is because it have
|
Beta Was this translation helpful? Give feedback.
-
Shame as But again, there’s no real CSV standard so we can’t complain. Sign. |
Beta Was this translation helpful? Give feedback.
-
I don’t see anything wrong in only accepting |
Beta Was this translation helpful? Give feedback.
-
Note: W3C also lists available "standads" at https://w3c.github.io/csvw/syntax/#standards. Unfortunately, all of them say |
Beta Was this translation helpful? Give feedback.
-
Back in 2019 there was still the discourse forum. Unfortunately, GitHub did not migrate the stuff correctly to the discussions, so the link above in the marked as resolved is dead. The thread had 100s of upvotes. Although the standard says it's comma, a semicolon is stil very common in Europe and default when exporting from e.g. German excel. |
Beta Was this translation helpful? Give feedback.
-
I found a very good instruction to change Excel's behavior at https://answers.microsoft.com/en-us/msoffice/forum/all/how-to-change-semicolon-to-comma-when-saving-csv/cf7c9537-82da-448d-9516-82f3a4b13fae. I know that it is not feasible to request that change from nearly the half of all Excel users of the world.
|
Beta Was this translation helpful? Give feedback.
-
Standards and would-be standards aside, I really don't understand what is so difficult about just doing it. Feels like people are rather opting into years of debate whether or not to be considerate about others, than into investing a few days of coding to be. Whole VS code extensions with that functionality (and more) have been created whilst this discussion was ongoing. So who is responsible, who could add that feature? A decent heuristic algorithm shouldn't be hard to develop. |
Beta Was this translation helpful? Give feedback.
-
In all honesty, it's an easy find & replace to correct CSVs with delimiters other than comma. |
Beta Was this translation helpful? Give feedback.
-
So is there a way to un-answer this and get this back on the radar again somehow? I don't often look at my CSV files up on Github, but that's at least partly due to the fact that they're all garbled. Are Europeans simply not allowed to have pretty formatted data on Github? |
Beta Was this translation helpful? Give feedback.
-
[I’d raise a bug but this seems to be the option I have, hope is correct tag for the Web App]
I have a CSV file that uses
;
as is common in EU countries that use,
as the currency separator. As far as I understand the CSV ‘specification’ that allows,
to be used in cells.mulberrysymbols/mulberry-symbols/blob/7279a34774d11af63ec14489573f538041c072d7/scripts/data/FinnishSymbolNames.csv
1;A;A;Aakkoset
2;a_-_lower_case;a;Aakkoset
3;above;yläpuolella;Kuvaileva sijainti
4;absent;poissa;Kuvaileva tila
5;acne_spots;akne;Terveys terveydentila
6;acorn;tammenterho;Kasvit ja puut
7;across;yli;Kuvaileva suunta
8;activity_centre;toimintakeskus;Vapaa-aika lelut
9;add;plus;Numerot laskutoimitukset
10;Advent;adventti;Uskonto juhlapäivä
11;Advent_calendar;adventtikalenteri;Uskonto yleiset
12;aerial_indoor;sisäantenni;Tekniikka TV
13;aerobics_dance;aerobic;Urheilu
14;aeroplane;lentokone;Liikenne ilmailu
15;afraid_lady;nainen peloissaan;Ihmiset tunteet
16;afraid_man;mies peloissaan;Ihmiset tunteet
17;after;perässä;Kuvaileva sijainti
18;afternoon;iltapäivä;Kuvaileva aika
19;aftershave;partavesi;Terveys hoitotarvikkeet
This file has been truncated. show original
However that seems to be breaking the GitHub pretty rendering of the CSV.
For these lines
Is this a CSV parsing bug.? A result of not having a real CSV standard? A localisation assumption bug? Or something else?
Can I work around it?
A hack is probably to convert the CSV to
,
separated and quoting strings with,
s but would rather keep as supplied to me by a contributor.Thanks
Steve
Beta Was this translation helpful? Give feedback.
All reactions