Skip to content

fix: use pull request head sha for diffs#148

Merged
43081j merged 1 commit intoe18e:mainfrom
hazre:codex/fix-pull-request-head-ref
Apr 11, 2026
Merged

fix: use pull request head sha for diffs#148
43081j merged 1 commit intoe18e:mainfrom
hazre:codex/fix-pull-request-head-ref

Conversation

@hazre
Copy link
Copy Markdown
Contributor

@hazre hazre commented Apr 10, 2026

Fixes the pull request head SHA lookup so fork PRs diff against the actual head commit. There was a typo in context2.payload.pull_reuqest?.head.sha.

I noticed it while trying to add the action in a codebase.

@github-actions
Copy link
Copy Markdown

⚠️ Duplicate Dependencies (found: 3, threshold: 1)

📦 Package 📋 Versions
@actions/http-client
2 versions
  • @e18e/action-dependency-diff@0.0.1
    • @actions/github@9.1.0
      • @actions/http-client@3.0.2

  • @e18e/action-dependency-diff@0.0.1
    • @actions/core@3.0.0
      • @actions/http-client@4.0.0

eslint-visitor-keys
2 versions
  • @e18e/action-dependency-diff@0.0.1
    • eslint@10.2.0
      • @eslint-community/eslint-utils@4.9.1
        • eslint-visitor-keys@3.4.3

  • @e18e/action-dependency-diff@0.0.1
    • eslint@10.2.0
      • eslint-visitor-keys@5.0.1

ignore
2 versions
  • @e18e/action-dependency-diff@0.0.1
    • eslint@10.2.0
      • ignore@5.3.2

  • @e18e/action-dependency-diff@0.0.1
    • typescript-eslint@8.58.1
      • @typescript-eslint/eslint-plugin@8.58.1
        • ignore@7.0.5

💡 To find out what depends on a specific package, run: npm ls example-package

@43081j 43081j merged commit 5d3c6ac into e18e:main Apr 11, 2026
4 checks passed
@hazre hazre deleted the codex/fix-pull-request-head-ref branch April 11, 2026 13:20
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