+1

On Tue, Mar 10, 2020, 23:09 Gerben <ger...@treora.com.invalid> wrote:

> +1
> On 10/03/2020 16:09, Benjamin Young wrote:
>
> +1 from me--with huge gratitude to Randall for making this all happen!! :D
>
> The signature and sha's all checked out fine. My local diff (on Windows) has 
> line ending differences (which I know about...per my git config). My diff 
> also shows the following files in the git checkout which are not in the 
> tarball--all of which I think are fine, but wanted to confirm and/or document 
> for future comparison/checks.
>
> Files in local git checkout which are not in the tarball include:
>  - .git/ (as for Randall)
>  - .nyc_output/ (probably due to a different reporter I used at some point)
>  - node_modules/ (we don't ship this in the tarball...nor should we afaik)
>  - packages/fragment-identifier/node_modules/ (same as above)
>  - packages/fragment-identifier/src/fragment.js (...not sure about this 
> one...)
>
> Everything else is identical (ignoring line endings--per platform 
> differences).
>
> Any thoughts (or explanations) about the fragment.js situation?
>
> fragment.js is not source code, but generated from fragment.pegjs. It is
> created when building/running the code, just like node_modules/ and other
> folders and files (I moreover see an esm/, lib/, LICENSE and NOTICE in each
> package folder).
>
> For an easier comparison, you could make a fresh clone of your repository
> into a new folder (e.g. run `git clone incubator-annotator/
> incubator-annotator-clean/`), and compare the sameness with that one
> instead. And then perhaps build and run it to validate whether it also
> works correctly.
>
> …
>
>

Reply via email to