[GitHub] [pulsar-dotpulsar] dionjansen commented on a change in pull request #67: Support for (optional) ack timeout and nack delay for consumers

[GitBox]

dionjansen commented on a change in pull request #67:
URL: https://github.com/apache/pulsar-dotpulsar/pull/67#discussion_r552553274



##########
File path: src/DotPulsar/Internal/InactiveUnackedMessageTracker.cs
##########
@@ -0,0 +1,31 @@
+namespace DotPulsar.Internal
+{
+    using System.Threading;

Review comment:
       Do you run code cleanup through VSC? I've used *Remove and sort using* 
but this orders it in the way I wrote it now.
   
   
![image](https://user-images.githubusercontent.com/5121647/103767382-cb9a6180-5020-11eb-8539-fa9df66d888a.png)
   
   But I can't find a code cleanup action in my IDE:
   
   
![image](https://user-images.githubusercontent.com/5121647/103767916-cab5ff80-5021-11eb-99a3-6fecdc340406.png)
   
   Also no mention in the 
[docs](https://docs.microsoft.com/en-us/visualstudio/mac/editor-behavior?view=vsmac-2019)
 from what I can find, are you using a different editor? For now I'll just keep 
the order by hand




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org