You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "mdedetrich (via GitHub)" <gi...@apache.org> on 2023/05/28 13:29:39 UTC

[GitHub] [incubator-pekko] mdedetrich opened a new pull request, #358: Add inline final to FutureConverters

mdedetrich opened a new pull request, #358:
URL: https://github.com/apache/incubator-pekko/pull/358

   While the inline isn't that important here, the most critical thing is that the addition of `final` in order to make sure that we don't get any problems down the road. Note that this change is making `FutureConverters` consistent with `OptionConverters`.


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org


[GitHub] [incubator-pekko] mdedetrich merged pull request #358: Add inline final to FutureConverters

Posted by "mdedetrich (via GitHub)" <gi...@apache.org>.
mdedetrich merged PR #358:
URL: https://github.com/apache/incubator-pekko/pull/358


-- 
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.

To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org