search

meetecho / janode

A Node.js adapter for the Janus WebRTC server
ISC License
98 stars 36 forks source link

Update package.json #30

Closed dota-coach closed 1 year ago

dota-coach commented 1 year ago

I could not compile, as there was not a main entry point defined in your package.json.

Would be great if you could add it.

atoppi commented 1 year ago

According to node docs

For new packages targeting the currently supported versions of Node.js, the "exports" field is recommended. For packages supporting Node.js 10 and below, the "main" field is required. If both "exports" and "main" are defined, the "exports" field takes precedence over "main" in supported versions of Node.js.

So I guess adding main is harmless since we already have exports.