Skip to content

fix: use base sha for PRs instead of ref - #177

Merged
43081j merged 1 commit into
mainfrom
jg/sha-ify-it
Aug 13, 2026
Merged

fix: use base sha for PRs instead of ref#177
43081j merged 1 commit into
mainfrom
jg/sha-ify-it

Conversation

@43081j

@43081j 43081j commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

The ref is a named point, like main, so it can point at a different
target to the PR (e.g. outdated branch).

Using the sha means we diff against the target commit rather than a
named ref.

The ref is a named point, like `main`, so it can point at a different
target to the PR (e.g. outdated branch).

Using the `sha` means we diff against the target commit rather than a
named ref.
@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.1
      • @actions/http-client@3.0.2

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

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

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

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

  • @e18e/action-dependency-diff@0.0.1
    • typescript-eslint@8.66.0
      • @typescript-eslint/eslint-plugin@8.66.0
        • ignore@7.0.6

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

@43081j
43081j merged commit 9a7f09f into main Aug 13, 2026
4 checks passed
@43081j
43081j deleted the jg/sha-ify-it branch August 13, 2026 16:44
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.

1 participant