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

use closure instead of block #602

Merged
merged 1 commit into from
Sep 12, 2023
Merged

use closure instead of block #602

merged 1 commit into from
Sep 12, 2023

Conversation

WindSoilder
Copy link
Contributor

No description provided.

@fdncred
Copy link
Collaborator

fdncred commented Sep 12, 2023

Does this fix the problem? The OP said changing to closure still produced some type of error. I don't use pueue so I'm not sure.

@fdncred
Copy link
Collaborator

fdncred commented Sep 12, 2023

let's land this anyway because block isn't supported. there could be other changes if the OP ever gets back to us in #600

@fdncred fdncred merged commit e61b94e into nushell:main Sep 12, 2023
@WindSoilder
Copy link
Contributor Author

WindSoilder commented Sep 12, 2023

Actually it works pretty fine for me. Yeah, let's wait for OP in #600 .

Actually there are some constraint to make it works with pueue. I'd like to add some Q&A session in relative README

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants