Mqtt.js: The IClientPublishOptions interface is missing the properties attribute

Created on 3 Apr 2020  路  2Comments  路  Source: mqttjs/MQTT.js

The IClientPublishOptions interface in the client-options.d.ts file is missing the properties attribute

image

bug question

All 2 comments

IClientPublishOptions

Awwww such Java/C# ish names

I'mma say this is a bug... we have no testing for typescript interfaces.... we should probably have testing for the typescript interfaces....

Also this opens up one of the spiciest topics in TypeScript naming conventions, "I" prefixing interfaces or not. See: https://github.com/microsoft/TypeScript-Handbook/issues/121 and also https://stackoverflow.com/questions/31876947/confused-about-the-interface-and-class-coding-guidelines-for-typescript/41967120#41967120

I'm not going to advocate for changing all the interfaces to non "I" prefixed, but we can definitely discuss it.

Was this page helpful?
0 / 5 - 0 ratings