-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add missing timeout property to type. #77
Conversation
@@ -10,6 +10,8 @@ declare class ProducerError extends Error { | |||
interface ProduceOptions extends amqp.Options.Publish { | |||
routingKey?: string; | |||
rpc?: boolean; | |||
/** Timeout in milliseconds for producing RPC request and waiting for the response. Does not affect non-rpc requests. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/** Timeout in milliseconds for producing RPC request and waiting for the response. Does not affect non-rpc requests. */ | |
// Timeout in milliseconds for producing RPC request and waiting for the response. Does not affect non-rpc requests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why comment needed, specifically for this field?
I suggest either document all properties, or none. Lets be consistent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The options are not intuitive, so I added docs to all of them.
@@ -10,6 +10,8 @@ declare class ProducerError extends Error { | |||
interface ProduceOptions extends amqp.Options.Publish { | |||
routingKey?: string; | |||
rpc?: boolean; | |||
/** Timeout in milliseconds for producing RPC request and waiting for the response. Does not affect non-rpc requests. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why comment needed, specifically for this field?
I suggest either document all properties, or none. Lets be consistent.
2662e2a
to
81b7bbf
Compare
No description provided.