Skip to content

Fix inaccurate $script: variable scope claims in two docs#56

Merged
HeyItsGilbert merged 1 commit intomainfrom
worktree-sharded-floating-honey
Apr 29, 2026
Merged

Fix inaccurate $script: variable scope claims in two docs#56
HeyItsGilbert merged 1 commit intomainfrom
worktree-sharded-floating-honey

Conversation

@HeyItsGilbert
Copy link
Copy Markdown
Member

@HeyItsGilbert HeyItsGilbert commented Apr 28, 2026

Summary

  • Corrects the claim in Outputs and Artifacts (Pattern 3) that $script: variables are only accessible within the same Invoke-psake call — they actually persist in the psake module scope across calls (verified against v5 source in discussion #374)
  • Adds a Properties-block gotcha to both affected pages: variables declared in a Properties block are silently overwritten on every Invoke-psake call because that block re-runs each time
  • Tightens the Best Practices Summary item 3 in Outputs and Artifacts to remove the ambiguous "not accessible outside psake" phrasing

Test plan

  • Review the rendered diff on both pages for clarity
  • Confirm the gotcha callout renders correctly as a blockquote

🤖 Generated with Claude Code

Corrects the claim in Outputs and Artifacts (Pattern 3) that $script:
variables are only accessible within the same Invoke-psake call — they
actually persist in the psake module scope across calls. Adds a
Properties-block gotcha to both affected pages: variables declared in
a Properties block are silently overwritten on every Invoke-psake call
because that block re-runs each time.

Closes https://github.com/orgs/psake/discussions/374

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 28, 2026

Deploy Preview for psake ready!

Name Link
🔨 Latest commit f7174a9
🔍 Latest deploy log https://app.netlify.com/projects/psake/deploys/69f0ed871242790008f2f396
😎 Deploy Preview https://deploy-preview-56--psake.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@HeyItsGilbert HeyItsGilbert enabled auto-merge (squash) April 28, 2026 18:38
@HeyItsGilbert HeyItsGilbert disabled auto-merge April 29, 2026 00:30
@HeyItsGilbert HeyItsGilbert merged commit 92978fe into main Apr 29, 2026
8 checks passed
@HeyItsGilbert HeyItsGilbert deleted the worktree-sharded-floating-honey branch April 29, 2026 00:31
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.

2 participants