Skip to content
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

Reinstate Disposal #149

Merged
merged 3 commits into from
Jun 2, 2023
Merged

Conversation

bartelink
Copy link
Member

@bartelink bartelink commented May 4, 2023

Fixes: #157

https://github.com/jet/equinox and propulsion both depend on the following taskSeq features

  • disposal via use! in task expressions
  • logging via try/finally

This PR reinstates this functionality, which accidentally got disabled in 27486f3

(edited in place to refer to the issue)

@bartelink
Copy link
Member Author

Tests pass on the desktop.
Would love to give a 0.3.1 or a 0.4.0-alpha.1 a test drive...

@abelbraaksma
Copy link
Member

abelbraaksma commented Jun 2, 2023

Thanks for testing this and our discussions on Slack ❤️!!! Sorry to leave this hanging so long, I kinda/sorta wanted to get other stuff baselined as well, but this is too important and should've been fixed already.

Merging now, publishing tomorrow.

@abelbraaksma
Copy link
Member

abelbraaksma commented Jun 2, 2023

Strange, I thought I can edit this with GH CodeSpaces. While it allows me to edit, I get a 403 when trying to commit to this PR. When I do it locally though, just check out this PR and commit & push, it "just works" ™️

(needed to update the release notes before merging)

release-notes.txt Outdated Show resolved Hide resolved
Co-authored-by: Ruben Bartelink <ruben@bartelink.com>
@abelbraaksma abelbraaksma merged commit f51aba7 into fsprojects:main Jun 2, 2023
@bartelink bartelink deleted the reinstate-disposal branch June 3, 2023 06:56
@abelbraaksma abelbraaksma added bug fix topic: taskseq-ce Related to the taskseq computation expression builders or overloads labels Oct 29, 2023
@abelbraaksma abelbraaksma added this to the v0.4.0-alpha.1 milestone Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix topic: taskseq-ce Related to the taskseq computation expression builders or overloads
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disposal is not called with use! and the like
2 participants