HI, I really love the tool and currently running it under windows. however my git diff file is around 300KB, the tool takes 3 hours to finish , without any output file (am using -F option). memory usage is around 800MB.
Just wondering if you have encountered the same issue before?

Tried diffy.org without no issues at all.
https://diffy.org/diff/4wng00ndqz7iudi

thanks. Travis diffReport.txt

when I try to run diff2html HEAD^ it opens the web browser and says it found 0 changes

when I try to run pretty-diff HEAD^ it shows me the correct diff

Not sure what's wrong but seems like something is broken

Platform: MacBook Pro (Retina, 15-inch, Mid 2014) macOS Sierra 10.12.6

I'm using:

var diff2html = require('diff2html');
...
        const parsed = diff2html.parse(changes);
        const html = diff2html.html(parsed, diff2html.defaultDiff2HtmlConfig);

Note: Diff content is already stored in variable: changes.

But cannot export html with diff content, only obtained empty template such as:

<div class="d2h-file-list-wrapper">
    <div class="d2h-file-list-header">
        <span class="d2h-file-list-title">Files changed (0)</span>
        <a class="d2h-file-switch d2h-hide">hide</a>
        <a class="d2h-file-switch d2h-show">show</a>
    </div>
    <ol class="d2h-file-list">
    
    </ol>
</div><div class="d2h-wrapper">
    
</div>

Also, tried docs example (with a .diff file exported) without success: diff2html -i file -- my-file-diff.diff

Kindly, can point to any direction?

diff2html handles modifications perfectly, but it seems to ignore both new files and deleted files.

I'm running diff2html via the cli. For reference: diff2html 2.3.0, diff2html-cli 2.5.0. I'm using this to diff between my working SVN directory and my web directory (where I've made several changes).

I've tried pipping the diff directly to diff2html and outputting the diff in a file and then using diff2html with file input, but the result is always the same. I'll get a perfect list/display of all my modified files, but nothing about deleted/new files.

To be sure, I checked the raw diff output, and I definitely see references to both deleted and new files: -- Only in /localpath/src/views: filename.php (for an added file) -- Only in /svnpath/src/views: anotherfile.php (for a removed file)

COMMAND: Pipping diff --exclude=.svn -ru /localpath /svpath | diff2html -F /var/www/html/diff-out.html -i stdin

COMMAND: By File diff --exclude=.svn -ru /localpath /svpath > diffOut.txt diff2html -F /var/www/html/diff-out.html -i file "diffOut.txt"

I tried removing the --exclude param, but I still don't see removed or new files in the diff2html output.

I looked around for examples of new and deleted files with diff2html and couldn't find any. I'm assuming diff2html supports deleted/new files because I saw an issue about ordering deleted/new files (github @ https://github.com/rtfpessoa/diff2html/issues/33 ), but I don't see anything about a command line argument to pass for diff2html-cli and was unable to find anyone else with this same problem.

Hi,

I encounter a git diff error when I run the following command using diff2html-cli :

diff2html -F my-pretty-diff.html --hwt ci/mail-template.html --ig ci -- --no-prefix -U1000 -M HEAD~1

I get this error :

Error: Command failed: git diff "--no-prefix" "-U1000" "-M" "HEAD~1" --no-color ":(exclude)ci"
fatal: ambiguous argument ':(exclude)ci': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

    at checkExecSyncError (node:child_process:681:11)
    at Object.execSync (node:child_process:718:15)
    at Object.execute (/usr/local/lib/node_modules/diff2html-cli/lib/utils.js:38:36)
    at runGitDiff (/usr/local/lib/node_modules/diff2html-cli/lib/cli.js:87:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/diff2html-cli/lib/cli.js:125:32)
    at step (/usr/local/lib/node_modules/diff2html-cli/lib/cli.js:63:23)
    at Object.next (/usr/local/lib/node_modules/diff2html-cli/lib/cli.js:44:53)
    at /usr/local/lib/node_modules/diff2html-cli/lib/cli.js:38:71
    at new Promise (<anonymous>)
    at __awaiter (/usr/local/lib/node_modules/diff2html-cli/lib/cli.js:34:12) {
  status: 128,
  signal: null,
  output: [
    null,
    <Buffer >,
    <Buffer 66 61 74 61 6c 3a 20 61 6d 62 69 67 75 6f 75 73 20 61 72 67 75 6d 65 6e 74 20 27 3a 28 65 78 63 6c 75 64 65 29 63 69 27 3a 20 75 6e 6b 6e 6f 77 6e 20 ... 143 more bytes>
  ],
  pid: 140,
  stdout: <Buffer >,
  stderr: <Buffer 66 61 74 61 6c 3a 20 61 6d 62 69 67 75 6f 75 73 20 61 72 67 75 6d 65 6e 74 20 27 3a 28 65 78 63 6c 75 64 65 29 63 69 27 3a 20 75 6e 6b 6e 6f 77 6e 20 ... 143 more bytes>
} []

Am I doing something wrong ? What am I missing ?

Version details :

• npm 7.5.1
• node 15.8.0
• diff2html 5.1.7

Hello again, this time a bug report:

Repro:

git clone https://github.com/mensfeld/broken-diff.git
cd https://github.com/mensfeld/broken-diff.git
./run.sh

browser will open and then scroll to file called web/assets/javascripts/application.js it looks as followed:

while on a really, really old version of this cli (2.7.0), it looks like so:

This is a sample start on what a TypeScript implementation would look like.

Discussion: https://github.com/rtfpessoa/diff2html-cli/issues/46

The key benefit of TypeScript here is strict typing of method arguments.

Other changes:

• using opn instead of open (deprecated)
• simpler export
• minor version bumps

⚠️ THIS IS NOT READY TO MERGE ⚠️

If TypeScript were to be implemented, the release procedures would need to change somewhat (it's the built files that need to be a part of the CLI, not the original TypeScript files).

If idea is approved, before release I would:

• add JSDoc
• add more strong typing of variables
• clean up scripts (e.g. yarn test would run yarn build first)
• figure out how to make the exported package still work 😅

In our internal application I am trying to pick up daily changes on few config files and send those diffs daily per email nicely formatted. I currently see that pull request: https://github.com/rtfpessoa/diff2html-cli/pull/38 which should make available to override some template options, but I am wondering: is jquery.js and highlight.js at all needed, when I want to produce nicely formatted diff files? I tested a few diffs and it makes no difference if I drop the js files.

Thanks.

└─[$] diff2html.cmd -i file -- difftest.diff [14:09:15] 'open' is not recognized as an internal or external command, operable program or batch file. child_process.js:507 throw err; ^

Error: Command failed: open /tmp/diff.html 'open' is not recognized as an internal or external command, operable program or batch file.

at checkExecSyncError (child_process.js:464:13)
at Object.execSync (child_process.js:504:13)
at Utils.runCmd (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\utils.js:58:25)
at Diff2HtmlInterface.preview (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\cli.js:106:11)
at onOutput (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\main.js:157:9)
at Diff2HtmlInterface.getOutput (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\cli.js:77:14)
at onInput (C:\Users\sreich\AppData\Roaming\npm\node_modules\diff2html-cli\src\main.js:145:7)
at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:380:3)

ideas on what's wrong?

Update all the packages to their latest versions.

~Note that yargs version 13.0.0 is out just not published to npm. I'm sure in a day or two it will be there: https://github.com/yargs/yargs/issues/1288 (after which we'll want to update the yarn.lock in this PR)~ 👌

~Note the minimum supported version of node has changed. Using anything lower than 8.12.0 will result in: error [email protected]: The engine "node" is incompatible with this module. Expected version ">= 8.12.0".~

@rtfpessoa -- please, either let me know which version you'd need to bump diff2html-cli to if this merges (or please change it yourself: https://github.com/rtfpessoa/diff2html-cli/blob/master/package.json#L3) 😅

update:

• yargs v13 still hadn't been published to npm so I have v12 in this PR.
• the codacity error I noted above doesn't seem to affect versions 6 and 7 so everything is fine 👌

Resolves #33 by allowing users to provide a path to a custom markup template when rendering output as html. More specifically:

• updated README.md with a list of variables that can be used within a custom template
• updated main.js with the -t, --template option to use a custom html
• updated cli.js to use template set in user config, otherwise defaults to existing template
• imported fs from cli.js to confirm existence of custom template, otherwise throws an error
• tests are passing

I realize this means your authorship may no longer appear in the generated output, so I fully understand if you're not comfortable accepting this PR. Thanks!