npm package includes 86MB file about geodesy HOT 10 CLOSED

Would definitely appreciate it if you could release a 1.1.2 that doesn't include that test file.

from geodesy.

Apologies for the delay in addressing this – I'm almost catching up with myself! v1.1.2 npm tarball is now 276KB.

from geodesy.

Thanks for bringing this to my attention.

I don't understand how those test data files could have ended up in the npm bundle. They are not in the git repo, and npm should follow .gitignore rules (which exclude the test data files, inter alia).

I shall have to investigate.

from geodesy.

Seems like your .gitignore file is missing, that might be why.

from geodesy.

I don't have a build process which generates files which need to be git-ignored, so there is no reason to check in my local .gitignore into the repo. npm publish should still be respecting that .gitignore file, though, just as Git itself does, so should not include the test data files in the npm package just as Git doesn't include them in the repo.

from geodesy.

I don't have a build process which generates files

npm install is a process that generates files.

npm publish won't be looking at root level .gitignore.

from geodesy.

npm install is a process that generates files

npm install is not part of a build process (it is not specified in package.json "scripts"), and also npm publish explicitly ignores node_modules.

The npm Developer Guide says "If there's no .npmignore file, but there is a .gitignore file, then npm will ignore the stuff matched by the .gitignore file".

from geodesy.

But you don't have a .gitignore file... so nothing is ignored.

from geodesy.

ping @chrisveness :­-)

from geodesy.

Great, thanks!

from geodesy.