nasa / ogma

Other
329 stars 24 forks source link

`ogma-core`: Remove dependency on `IfElse` #150

Closed ivanperez-keera closed 6 hours ago

ivanperez-keera commented 3 days ago

Description

The dependency on IfElse is quite unnecessary: it seems like the only function we use from that library is awhen :: Monad m => Maybe a -> (a -> m ()) -> m (), which is a type-specialized version of Data.Foldable.for_.. Since the latter is in base, we can simplify Ogma by removing the dependency on IfElse.

Type

Additional context

None.

Requester

Method to check presence of bug

Not applicable (not a bug).

Expected result

IfElse is not required by any library in ogma.

Desired result

IfElse is not required by any library in ogma.

Proposed solution

Replace calls to awhen with calls to for_

Remove the dependency on IfElse.

Further notes

None.

ivanperez-keera commented 17 hours ago

Change Manager: Confirmed that the issue exists.

ivanperez-keera commented 16 hours ago

Technical Lead: Confirmed that the issue should be addressed.

ivanperez-keera commented 16 hours ago

Technical Lead: Issue scheduled for fixing in Ogma 1.4.1. Fix assigned to: @ivanperez-keera.

ivanperez-keera commented 6 hours ago

Implementor: Solution implemented, review requested.

ivanperez-keera commented 6 hours ago

Change Manager: Verified that:

ivanperez-keera commented 6 hours ago

Change Manager: Implementation ready to be merged.