-
Notifications
You must be signed in to change notification settings - Fork 257
Controlling the CLI through environment variables #16595
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
base: master
Are you sure you want to change the base?
Conversation
Documentation ReviewI've reviewed this blog post for PR #16595. Here are my findings: Critical Issues1. Missing frontmatter (line 1)
The file should start with proper frontmatter like: ---
title: "Controlling the CLI Through Environment Variables"
date: 2025-01-XX
meta_image: meta.png
authors:
- i-am-tom
tags:
- features
---2. Missing trailing newline 3. Incorrect heading level (line 2) 4. Title case issue (line 2) Style Issues5. Sentence case for H2 headings (lines 6, 17, 26, 44)
Actually, all H2 headings are already correct - nice work! 6. Missing 7. Link text clarity (line 10) 8. Code fence language (line 30) Content Suggestions9. Code example completeness 10. Links verification
Positive Notes
SummaryThis is a well-written blog post with clear technical content. The main issue is the missing frontmatter, which is required for Hugo to process the blog post correctly. Once the frontmatter is added and the minor formatting issues are addressed, this will be ready to publish. Action Required: Please add the required frontmatter fields at the top of the file, remove the duplicate H1 heading, ensure the file ends with a newline, and consider adding a If you'd like me to make these changes or need any clarification, mention @claude and I'll be happy to help! |
86e28ae to
62cecb0
Compare
|
Your site preview for commit 1f940af is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-16595-1f940afe.s3-website.us-west-2.amazonaws.com. |
A blog post for the CLI environment variable changes.