Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.57%. Comparing base (94b8d9a) to head (ef66172).
Report is 4 commits behind head on main.

@@           Coverage Diff           @@
##             main     #122   +/-   ##
=======================================
  Coverage   95.57%   95.57%           
=======================================
  Files          11       11           
  Lines         769      769           
=======================================
  Hits          735      735           
  Misses         31       31           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Hey @tech-voker,

Great catch 👍 . Since Result is being exported, renaming it would be a breaking change in packages using it explicitly. I think we can merge this as long as we keep the Result interface declared in future.go alongside the renamed ResultTask interface.
It would be nice if you could also add a specific test for this scenario as well. For instance, verifying that assigning the result of Submit() ResultTask[O] to a Result variable would work.
A deprecation comment needs to be added to Result as well for clarity.

Hey @tech-voker,

Great catch 👍 . Since Result is being exported, renaming it would be a breaking change in packages using it explicitly. I think we can merge this as long as we keep the Result interface declared in future.go alongside the renamed ResultTask interface. It would be nice if you could also add a specific test for this scenario as well. For instance, verifying that assigning the result of Submit() ResultTask[O] to a Result variable would work. A deprecation comment needs to be added to Result as well for clarity.

Hi @alitto I have introduced the changes as suggested. Let me know if you need anything else to be covered in tests.