rtfpessoa/diff2html-cli

Custom html diff to send per email

dpetrov opened this issue · 6 comments

dpetrov commented

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: #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.

tylerjpeterson commented

@dpetrov - in my limited experience, neither are technically required to output a nice looking diff. I removed those scripts, the associated CSS, and the JS instantiation from the template without any issues. Pasted my bare-bones template below if it helps...

<!doctype html>
<html lang="en">
<head>
	<meta charset="utf-8">
	<title></title>
	<!--diff2html-css-->
</head>
<body style="text-align: center; font-family: 'Source Sans Pro',sans-serif;">
	<div id="diff">
		<!--diff2html-diff-->
	</div>
</body>
</html>
dpetrov commented

Thanks @tylerjpeterson - it's much appreciated. How do you override the currently templates, before #38 goes live? Do you override it globally?

rtfpessoa commented
tylerjpeterson commented

@dpetrov - I've installed my fork directly via GitHub until it's accepted/denied.

$ npm i github:tylerjpeterson/diff2html-cli
  • 👍1
dpetrov commented

Thanks both of you, it's much appreciated. --htmlWrapperTemplate works nicely.

  • 👍2