You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The parser swallows the timestamps as part of the filename, too, which (1) is distracting clutter and (2) breaks syntax highlighting since it can't interpret the file type from the suffix anymore.
getFilename in diff-parser.js should be able to strip off these suffixes. I think they're always rendered with a \t character between the filename and the time, so it should be pretty straightforward.
The text was updated successfully, but these errors were encountered:
I was actually reading the generated installed JS and didn't notice the comment in the TS source that the regex at the end is already intended to strip off these dates.
So update on the above issue: I'm not immediately sure why, but this doesn't work at least with /usr/bin/diff on macOS, which outputs a header exactly as shown in the issue.
If I want to diff plain files, independent of git, the natural thing to do is something like:
diff -u file1.js file2.js | diff2html -i stdin
This generally works, but it does not parse the filenames correctly.
diff
by default renders the filename header like this:The parser swallows the timestamps as part of the filename, too, which (1) is distracting clutter and (2) breaks syntax highlighting since it can't interpret the file type from the suffix anymore.
getFilename
indiff-parser.js
should be able to strip off these suffixes. I think they're always rendered with a\t
character between the filename and the time, so it should be pretty straightforward.The text was updated successfully, but these errors were encountered: