-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ogma-core
: Remove dependency on IfElse
#150
Labels
CR:Status:Closed
Admin only: Change request that has been completed
CR:Type:Management
Admin only: Change request for conformance with policies or procedures
Milestone
Comments
ivanperez-keera
added
CR:Status:Initiated
Admin only: Change request that has been initiated
CR:Type:Management
Admin only: Change request for conformance with policies or procedures
labels
Sep 22, 2024
Change Manager: Confirmed that the issue exists. |
ivanperez-keera
added
CR:Status:Confirmed
Admin only: Change request that has been acknowledged by the change manager
and removed
CR:Status:Initiated
Admin only: Change request that has been initiated
labels
Sep 22, 2024
Technical Lead: Confirmed that the issue should be addressed. |
ivanperez-keera
added
CR:Status:Accepted
Admin only: Change request accepted by technical lead
and removed
CR:Status:Confirmed
Admin only: Change request that has been acknowledged by the change manager
labels
Sep 22, 2024
Technical Lead: Issue scheduled for fixing in Ogma 1.4.1. |
ivanperez-keera
added
CR:Status:Scheduled
Admin only: Change requested scheduled
and removed
CR:Status:Accepted
Admin only: Change request accepted by technical lead
labels
Sep 22, 2024
ivanperez-keera
added
CR:Status:Implementation
Admin only: Change request that is currently being implemented
and removed
CR:Status:Scheduled
Admin only: Change requested scheduled
labels
Sep 22, 2024
ivanperez-keera
changed the title
Remove dependency on
Sep 22, 2024
IfElse
ogma-core
: Remove dependency on IfElse
ivanperez-keera
added a commit
that referenced
this issue
Sep 22, 2024
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. This commit replaces uses of awhen with uses of for_.
ivanperez-keera
added a commit
that referenced
this issue
Sep 22, 2024
All uses of functions from IfElse have been replaced with uses of functions from base. This commit removes the dependency on IfElse from the ogma-core's cabal file.
ivanperez-keera
added a commit
that referenced
this issue
Sep 22, 2024
ivanperez-keera
added a commit
that referenced
this issue
Sep 22, 2024
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. This commit replaces uses of awhen with uses of for_.
ivanperez-keera
added a commit
that referenced
this issue
Sep 22, 2024
All uses of functions from IfElse have been replaced with uses of functions from base. This commit removes the dependency on IfElse from the ogma-core's cabal file.
ivanperez-keera
added a commit
that referenced
this issue
Sep 22, 2024
ivanperez-keera
added a commit
to ivanperez-keera/ogma
that referenced
this issue
Sep 22, 2024
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. This commit replaces uses of awhen with uses of for_.
ivanperez-keera
added a commit
to ivanperez-keera/ogma
that referenced
this issue
Sep 22, 2024
All uses of functions from IfElse have been replaced with uses of functions from base. This commit removes the dependency on IfElse from the ogma-core's cabal file.
ivanperez-keera
added a commit
to ivanperez-keera/ogma
that referenced
this issue
Sep 22, 2024
Implementor: Solution implemented, review requested. |
ivanperez-keera
added
CR:Status:Verification
Admin only: Change request that is currently being verified
and removed
CR:Status:Implementation
Admin only: Change request that is currently being implemented
labels
Sep 22, 2024
Change Manager: Verified that:
|
Change Manager: Implementation ready to be merged. |
ivanperez-keera
added
CR:Status:Closed
Admin only: Change request that has been completed
and removed
CR:Status:Verification
Admin only: Change request that is currently being verified
labels
Sep 22, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
CR:Status:Closed
Admin only: Change request that has been completed
CR:Type:Management
Admin only: Change request for conformance with policies or procedures
Description
The dependency on
IfElse
is quite unnecessary: it seems like the only function we use from that library isawhen :: Monad m => Maybe a -> (a -> m ()) -> m ()
, which is a type-specialized version ofData.Foldable.for_
.. Since the latter is inbase
, we can simplify Ogma by removing the dependency onIfElse
.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 tofor_
Remove the dependency on IfElse.
Further notes
None.
The text was updated successfully, but these errors were encountered: