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

Adds TPL based runTask to Deployment #3858

Merged
merged 2 commits into from
May 1, 2020

Conversation

yreynhout
Copy link
Contributor

Not entirely sure how useful this is to have in the library, but it's something I keep embedding in my own codebase.

@mikhailshilkov
Copy link
Member

Hi @yreynhout I haven't looked at your PR before because it is marked as draft. Do you still want to do something here, or is it done? I think it's a useful change.

@yreynhout yreynhout marked this pull request as ready for review April 30, 2020 15:28
@yreynhout
Copy link
Contributor Author

Hi @mikhailshilkov - pretty much done from my point of view - just didn't know whether it was deemed a useful addition or not. Your call to be honest.

@mikhailshilkov
Copy link
Member

mikhailshilkov commented Apr 30, 2020

Great! Do you mind rebasing on the latest and adding a short note to CHANGELOG.md?

@yreynhout
Copy link
Contributor Author

Sure, no problem

@yreynhout
Copy link
Contributor Author

@mikhailshilkov done.

Copy link
Member

@mikhailshilkov mikhailshilkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@mikhailshilkov mikhailshilkov merged commit f36a8b4 into pulumi:master May 1, 2020
@yreynhout yreynhout deleted the deployment-runtask branch October 22, 2020 10:30
abhinav pushed a commit to pulumi/pulumi-dotnet that referenced this pull request Jan 11, 2023
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.

None yet

2 participants