search

snowleopard / selective

Selective Applicative Functors: Declare Your Effects Statically, Select Which to Execute Dynamically
MIT License
202 stars 21 forks source link

Drop redundant constraint in `selectT` #56

Closed snowleopard closed 2 years ago

snowleopard commented 2 years ago

As pointed out by @turion in #53, we don't need it!

snowleopard commented 2 years ago

@turion I've invited you to join as a collaborator so you can review PRs. Hope you don't mind!

turion commented 2 years ago

@turion I've invited you to join as a collaborator so you can review PRs. Hope you don't mind!

Thanks a lot! Although, for reviewing I don't need to be collaborator. If you want me give me commit rights, it makes sense to add me.

snowleopard commented 2 years ago

for reviewing I don't need to be collaborator

I don't think that's true: I couldn't add you as a reviewer for this PR before you joined as a collaborator.

Anyway, thanks for the review!