[Pkg-javascript-devel] [RFR] highlight.js and impress.js

[Jérémy Lal]

On 11/12/2012 23:42, Cédric Boutillier wrote:
> 
> Dear JavaScript team members,
> 
> I would like to kindly ask you to review my first two JavaScript
> packages to see if they are conform to what is expected. Those are
> dependencies of a Ruby tool, mdpress, used to create presentations from
> Markdown files.
> 
> The two packages are:
> 
> - impress.js
>    This one is very simple, and I hope I got it right (at least almost).
> 
> - highlight.js
>    Here I am less confident, especially about the node-highlight binary
>    package: I ensured that I install the same files as what I would get
>    from the command 'npm install highlight.js'. However, since I am
>    illiterate in nodejs business, I could not check if it was going to
>    work right. In particular would the .css files shipped in the
>    libjs-highlight package be useful for the nodejs-highlight package?
> 
>    Also would you recommend to build the documentation? what would be
>    the name of the -doc package?
> 
> Thanks in advance for your comments.
> 
> The packages can be found in the team repository:
> git://anonscm.debian.org/git/pkg-javascript/impress.js.git
> git://anonscm.debian.org/git/pkg-javascript/highlight.js.git/

Hi Cédric,

Yes it is always better to generate documentation.
If it's a small set of (generated) files please just include
them in each binary package.

_Finding a good naming policy for -doc packages would be nice_

A couple of remarks about highlight.js:

* node-highlight must depend on nodejs, not recommend it.
* please provide README.md in both packages

For testing highlight with nodejs, install that package,
type "nodejs" in a terminal, and run :

  var hl = require('highlight'); // or './highlight' if you're in the build dir
  hl.highlightAuto('int myint = 1;').value

Cheers,
Jérémy.