Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Handle PascalCase PR event payload keys#4

Merged
Marius Storhaug (MariusStorhaug) merged 2 commits into
mainPSModule/Get-PSModuleSettings:mainfrom
fix-event-payload-casingPSModule/Get-PSModuleSettings:fix-event-payload-casingCopy head branch name to clipboard
Jan 17, 2026
Merged

Handle PascalCase PR event payload keys#4
Marius Storhaug (MariusStorhaug) merged 2 commits into
mainPSModule/Get-PSModuleSettings:mainfrom
fix-event-payload-casingPSModule/Get-PSModuleSettings:fix-event-payload-casingCopy head branch name to clipboard

Conversation

@MariusStorhaug

@MariusStorhaug Marius Storhaug (MariusStorhaug) commented Jan 17, 2026

Copy link
Copy Markdown
Member
  • Handle Action/PullRequest casing in event payloads when determining PR action and merged status.

Fixes

Testing

  • Not run (logic-only change).

Copilot AI review requested due to automatic review settings January 17, 2026 16:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses case-sensitivity issues in GitHub event payload parsing by adding support for PascalCase property names (Action and PullRequest) alongside the existing lowercase variants (action and pull_request).

Changes:

  • Added fallback checks for PascalCase Action property when determining pull request action
  • Added fallback checks for PascalCase PullRequest property and extracted it to a dedicated variable for cleaner code
  • Refactored the merged status check to use the extracted pullRequest variable

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/main.ps1 Outdated
@MariusStorhaug Marius Storhaug (MariusStorhaug) merged commit 5be8144 into main Jan 17, 2026
15 checks passed
@MariusStorhaug Marius Storhaug (MariusStorhaug) deleted the fix-event-payload-casing branch January 17, 2026 17:05
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.

Handle PullRequest/Action casing in event payloads to detect merged PRs

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.