Email collation of PR comments
    David Holmes 
    david.holmes at oracle.com
       
    Tue Jan 18 07:20:07 UTC 2022
    
    
  
Not sure if something was recently changed but I'm now noticing some 
emails from PR's are collating comments in a way that makes the email 
very awkward to read/understand.
For example in:
https://github.com/openjdk/jdk/pull/7079
I made two comments (one stand-alone and one as part of approving the 
change):
------------
dholmes-ora commented 11 minutes ago
Thanks Thomas! I was too focused on the original string and didn't check 
that the exception code was going to make a copy anyway. There's likely 
multiple copies made on the way to the final String object.
dholmes-ora approved these changes 10 minutes ago
dholmes-ora left a comment
Seems fine.
Thanks,
David
------------
but the email that was sent flipped the order and just ran them 
together, so the email has:
------------
Seems fine.
Thanks,
David
Thanks Thomas! I was too focused on the original string and didn't check 
that the exception code was going to make a copy anyway. There's likely 
multiple copies made on the way to the final String object.
-------------
I would have expected the comments to be presented in the order in which 
they were written, and that there would be some kind of delimiter 
between them so you could tell they were distinct comments.
Cheers,
David
    
    
More information about the skara-dev
mailing list