Skip to content

fix: improve variable references#7888

Closed
FilippoBau wants to merge 2 commits intopagopa:masterfrom
FilippoBau:master
Closed

fix: improve variable references#7888
FilippoBau wants to merge 2 commits intopagopa:masterfrom
FilippoBau:master

Conversation

@FilippoBau
Copy link
Copy Markdown

@FilippoBau FilippoBau commented Feb 24, 2026

Short description

This Pull Request updates the PR title validation workflow to align with current GitHub Actions best practices. It improves how inputs are passed to scripts and applies the principle of least privilege to the GitHub token for better overall repository hygiene.

List of changes proposed in this pull request

  • Refactored parameter passing to use secure environment variables instead of inline string interpolation.
  • Added an explicit permissions: pull-requests: write block to restrict the GITHUB_TOKEN to the minimum necessary scope.
  • Updated gh api calls to use environment variables ($REPO, $PR_NUMBER), ensuring compliance with GitHub Actions robustness guidelines.

@dmnplb
Copy link
Copy Markdown
Contributor

dmnplb commented Apr 13, 2026

Hi @FilippoBau and thank you for your contribution.
The file references have changed in the meantime, so I have opened a new PR:

Please check that the reapplied changes are technically precise and match the original intention.

github-merge-queue Bot pushed a commit that referenced this pull request Apr 13, 2026
## Short description
This PR reapplies the changes proposed by @FilippoBau in the following
PR:
* #7888

Please refer to the PR original description for the tecnical details.
@dmnplb dmnplb closed this Apr 13, 2026
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