Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Update inputs/outputs doc #3003

Merged
merged 20 commits into from
Jun 15, 2023
Merged

Conversation

susanev
Copy link
Contributor

@susanev susanev commented Jun 7, 2023

Reverts #3002

@pulumi-bot
Copy link
Collaborator

Your site preview for commit 2248df9 is ready! 🎉

http://pulumi-hugo-origin-pr-3003-2248df98.s3-website.us-west-2.amazonaws.com.

@susanev susanev requested a review from lukehoban June 8, 2023 00:36
Copy link
Member

@jkodroff jkodroff left a comment

Choose a reason for hiding this comment

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

This is a huge improvement! Thanks so much for taking the time to do this!

@pulumi-bot
Copy link
Collaborator

Your site preview for commit 176b1be is ready! 🎉

http://pulumi-hugo-origin-pr-3003-176b1be0.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit ef999e6 is ready! 🎉

http://pulumi-hugo-origin-pr-3003-ef999e67.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit 9ff6fab is ready! 🎉

http://pulumi-hugo-origin-pr-3003-9ff6fab4.s3-website.us-west-2.amazonaws.com.


However, this pattern isn't possible because resource names need to be available and known at runtime in order to construct the URN of the resource.

If you really wish to pass an output from one resource as the resource name of another resource, you will need to use a {{< pulumi-apply >}}.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think using {{< pulumi-apply >}} is not working outside of choosers, I get a broken link. Should we use [apply](#apply) in this document?

@pulumi-bot
Copy link
Collaborator

Your site preview for commit 1a272de is ready! 🎉

http://pulumi-hugo-origin-pr-3003-1a272de8.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit dbc1d89 is ready! 🎉

http://pulumi-hugo-origin-pr-3003-dbc1d897.s3-website.us-west-2.amazonaws.com.

@jaxxstorm jaxxstorm changed the title Revert "Revert "pass at updating inputs/outputs doc"" Update inputs/outputs doc Jun 8, 2023
@pulumi-bot
Copy link
Collaborator

Your site preview for commit f76857e is ready! 🎉

http://pulumi-hugo-origin-pr-3003-f76857ed.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit 3ed4762 is ready! 🎉

http://pulumi-hugo-origin-pr-3003-3ed47623.s3-website.us-west-2.amazonaws.com.

@DSoko2
Copy link

DSoko2 commented Jun 9, 2023

This is a great improvement! I think the "Outputs" section could be even a bit more accessible for some programmers by rearranging it a bit:

I would consider moving accessing plain values from outputs (the last par and bullet point list) up and the technical details about dependency tracking (staring from "Outputs are als how Pulumi tracks...") down. From my insight, often the first time people think about what Output is and looking it up is when they notice they cannot just access the plain value of properties on resources.

You may consider removing the "monad" reference. I feel like the word "monad" makes many (not only novice) developers feel uncomfortable as they do not have a firm understanding of what it means. It also does not add much here because the hint towards promises tells people that know about monads that Output is most likely monad-like, too. Some programmers may not read on in the section after bumping into the monad reference.

"Promises" is well known in JS/TS/Web stetting, developers in other environments may not know it well or would have an easier time with "Future". Perhaps putting the async fact first and then appending "like Futures or Promises" may be a solution.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit bc7c27c is ready! 🎉

http://pulumi-hugo-origin-pr-3003-bc7c27c5.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit 0a21d4a is ready! 🎉

http://pulumi-hugo-origin-pr-3003-0a21d4a5.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit f53c73a is ready! 🎉

http://pulumi-hugo-origin-pr-3003-f53c73ac.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit 2b1379b is ready! 🎉

http://pulumi-hugo-origin-pr-3003-2b1379b1.s3-website.us-west-2.amazonaws.com.

@pulumi-bot
Copy link
Collaborator

Your site preview for commit c9c1981 is ready! 🎉

http://pulumi-hugo-origin-pr-3003-c9c1981c.s3-website.us-west-2.amazonaws.com.

@jaxxstorm jaxxstorm merged commit ddeabcf into master Jun 15, 2023
@jaxxstorm jaxxstorm deleted the revert-3002-revert-2996-inputs_outputs branch June 15, 2023 18:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants