Update attributions-check.sh

[Mmansoor89]

Description

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

---

Note

Harden the attributions check script with strict mode and clearer logging/error guidance.

• Scripts:
  • Hardens development/attributions-check.sh with strict mode (set -euo pipefail).
  • Adds clear step logging and a final success message.
  • Enhances failure path: detailed stderr guidance when attribution.txt is stale.
  • Minor refactor: centralizes ATTRIBUTIONS_FILE var and organizes flow.

^{Written by Cursor Bugbot for commit 4dfca9a. This will update automatically on new commits. Configure here.}

[github-actions]

CLA Signature Action:

Thank you for your submission, we really appreciate it. We ask that you read and sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just by adding a comment to this pull request with this exact sentence:

I have read the CLA Document and I hereby sign the CLA

By commenting with the above message you are agreeing to the terms of the CLA. Your account will be recorded as agreeing to our CLA so you don't need to sign it again for future contributions to this repository.

0 out of 1 committers have signed the CLA.
❌ @Mmansoor89