Inconsistent type for port field about node-consul

[silas]

If you send a number to consul options it should work fine assuming you're not sending 0, does that not work? What do you suggest changing, just setting the || '8500' to a number of something?

[TobyColeman]

Sorry, I should have probably clarified that this is more an issue when using this library with Typescript:

import * as Consul from 'consul'

const myConsul: Consul = new Consul({ port: '8500' })
myConsul.agent.service.register({ name: 'redis', port: 8500 })

Of course, passing either a number or string is fine if you're just using JS :).

I suggest maybe just updating the docs to clarify that port should be a number in both cases, and perhaps make the default port a number like you said?