m59peacemaker/browser-insular-observer

browser-insular-observer's Issues

The published package is broken

Hi,

first, sorry it took me this long to notice, I was occupied elsewhere.

It appears that the package published to the npm registry does not include the dist directory, making it unusable.

I did a dry run on npm publish to see if we have misconfigured something, but everything seems fine:

npm notice 📦  [email protected]
npm notice === Tarball Contents === 
npm notice 6.6kB LICENSE            
npm notice 1.2kB dist/index.js      
npm notice 1.4kB package.json       
npm notice 887B  dist/index.d.ts.map
npm notice 1.2kB dist/index.js.map  
npm notice 2.2kB README.md          
npm notice 1.1kB dist/index.d.ts    
npm notice 2.3kB index.ts           
npm notice 218B  .travis.yml        
npm notice === Tarball Details === 
npm notice name:          insular-observer                        
npm notice version:       0.1.0                                   
npm notice package size:  5.9 kB                                  
npm notice unpacked size: 17.1 kB                                 
npm notice shasum:        b367770edaaccd55de44e505558d6976184cf66f
npm notice integrity:     sha512-ddHVnBnhOxCAf[...]En55Q0kOhTLgQ==
npm notice total files:   9                                       
npm notice

Could you please look into this? Please try running npm publish --dry-run yourself first to see if the dist directory would be included if publishing from your machine. Also - are you using npm or yarn? If you're using yarn to publish the package, I'm guessing yarn might not be compatible with some aspect of our setup here.

TypeScript support

Hello,

would you be willing to support TypeScript ecosystem? There are various options, however, I think it would be best for future maintainability to port the code to TypeScript instead of adding separate type declaration files.

I am willing to do all the work myself and send a pull request.

Would that be OK with you, or would you rather go for a different option?