Skip to content

Fix JSON syntax in PULUMI_CONFIG example #15512

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

adamgordonbell
Copy link
Contributor

Summary

  • Fixed invalid JSON syntax in the PULUMI_CONFIG environment variable example
  • Changed single quotes to double quotes to make it valid JSON

Problem

The current example in the environment variables documentation shows:

PULUMI_CONFIG="{'project:myTag':'val1','project:mySecret':'val2'}"

This causes a JSON parsing error: Expecting property name enclosed in double quotes

Solution

Updated to use valid JSON syntax:

PULUMI_CONFIG='{"project:myTag":"val1","project:mySecret":"val2"}'

Test plan

  • Verified the JSON syntax is now valid
  • Linting and formatting checks pass

Fixes #15490

🤖 Generated with Claude Code

Changed the PULUMI_CONFIG environment variable example to use valid JSON
syntax with double quotes around property names and values. The previous
example used single quotes which caused JSON parsing errors.

Fixes #15490

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@pulumi-bot
Copy link
Collaborator

Added clarification that PULUMI_CONFIG must be valid JSON format
even when ignored during normal operations, to prevent JSON parsing
errors that users encounter when accidentally setting invalid JSON.

Updates #15490

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@pulumi-bot
Copy link
Collaborator

@pulumi-bot
Copy link
Collaborator

@adamgordonbell
Copy link
Contributor Author

Fixes #15490

Copy link
Contributor

@thoward thoward left a comment

Choose a reason for hiding this comment

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

Nice job! We really appreciate the contribution.

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.

Incorrect JSON quoting in PULUMI_CONFIG example on Environment Variables page
3 participants