Revamped TF vs. Pulumi IaC page #14817
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ericrudder
reviewed
Apr 23, 2025
asafashirov
reviewed
Apr 23, 2025
asafashirov
reviewed
Apr 23, 2025
asafashirov
reviewed
Apr 23, 2025
asafashirov
reviewed
Apr 23, 2025
asafashirov
reviewed
Apr 24, 2025
asafashirov
reviewed
Apr 24, 2025
asafashirov
reviewed
Apr 24, 2025
asafashirov
reviewed
Apr 24, 2025
asafashirov
reviewed
Apr 24, 2025
asafashirov
reviewed
Apr 24, 2025
| 🔧 **Terraform** | ||
|
|
||
| ## Get Started with Pulumi | ||
| - ❌ Proprietary domain specific language (HCL) |
Contributor
There was a problem hiding this comment.
Feedback from Eric: let's not use ANY EMOJIS in our content
Contributor
There was a problem hiding this comment.
What's a better way of communicating this?
Contributor
Author
There was a problem hiding this comment.
Gotcha. Emojis were in the variant page, so I pulled them into this one. Happy to get rid of them if that's our standard. Perhaps that should be in the style guide?
Contributor
There was a problem hiding this comment.
Good call; here's the PR for the style guide: #14833
Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com>
Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com>
Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com>
Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com>
Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com>
Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com>
Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com>
Co-authored-by: Eric Rudder <ericrudder@users.noreply.github.com>
asafashirov
approved these changes
Apr 24, 2025
Contributor
asafashirov
left a comment
asafashirov
left a comment
There was a problem hiding this comment.
lgtm
my only concern is potentially dropping for "pulumi vs terraform" search queries; but if that does happen, it should be temporary because google will eventually understand the company >< product entity relationship.
thoward
approved these changes
Apr 25, 2025
Contributor
thoward
left a comment
thoward
left a comment
There was a problem hiding this comment.
Nice job! We really appreciate the contribution.
thoward
pushed a commit
that referenced
this pull request
May 15, 2025
* revamped TF vs. pulumi IaC page * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> * Update content/docs/iac/concepts/vs/terraform/_index.md Co-authored-by: Eric Rudder <ericrudder@users.noreply.github.com> * Incorporates reviewer feedback [pull 14817] --------- Co-authored-by: asaf <69985318+asafashirov@users.noreply.github.com> Co-authored-by: Eric Rudder <ericrudder@users.noreply.github.com>
9 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
This revamp does the following:
This page will be used in upcoming Pulumi vs. Terraform campaigns and this PR lays the groundwork for its content.
As discussed @thoward , I'm finding an appropriate reviewer and will tag them in when ready.