Skip to content

Pin commit in github Actions#244

Merged
thomas-e-alma merged 1 commit into
developfrom
feature/ecom-3989-security-pin-version-in-github-action-in-php-client2
Mar 25, 2026
Merged

Pin commit in github Actions#244
thomas-e-alma merged 1 commit into
developfrom
feature/ecom-3989-security-pin-version-in-github-action-in-php-client2

Conversation

@Francois-Gomis

@Francois-Gomis Francois-Gomis commented Mar 24, 2026

Copy link
Copy Markdown
Collaborator

Reason for change

https://linear.app/almapay/issue/ECOM-3989/security-pin-version-in-github-action-in-php-client

Seccurity request

Code changes

How to test

As a reviewer, you are encouraged to test the PR locally.

Checklist for authors and reviewers

  • The title of the PR uses business wording, not technical jargon, for the changelog readers to understand it
  • The PR implements the changes asked in the referenced task / issue
  • The automated tests are compliant with the testing strategy
  • The tests are relevant, and cover the corner/error cases, not only the happy path
  • You understand the impact of this PR on existing code/features
  • The changes include adequate logging and Datadog traces
  • Documentation is updated (API, developer documentation, ADR, Notion...)

Non applicable

@Francois-Gomis Francois-Gomis requested a review from a team as a code owner March 24, 2026 16:51
@Francois-Gomis Francois-Gomis added skip-changelog Exclude this change from the next release changelog type: security and removed type: feature labels Mar 24, 2026
@thomas-e-alma

Copy link
Copy Markdown
Contributor

=== merging with requirements not being met ===

due to the CI current state and short delays, this PR will be merged as-is.

@thomas-e-alma thomas-e-alma merged commit e2a9883 into develop Mar 25, 2026
8 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Exclude this change from the next release changelog type: security

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants