Skip to content

Actually check committed generated files#10596

Merged
gilles-peskine-arm merged 1 commit into
Mbed-TLS:developmentfrom
gilles-peskine-arm:check_committed_generated_files-mbedtls-actually_check_mbedtls
Feb 12, 2026
Merged

Actually check committed generated files#10596
gilles-peskine-arm merged 1 commit into
Mbed-TLS:developmentfrom
gilles-peskine-arm:check_committed_generated_files-mbedtls-actually_check_mbedtls

Conversation

@gilles-peskine-arm

Copy link
Copy Markdown
Contributor

We were accidentally running the check in TF-PSA-Crypto instead of in Mbed TLS.

Necessary for Mbed-TLS/TF-PSA-Crypto#690.

PR checklist

  • changelog not required because: test only
  • development PR here
  • TF-PSA-Crypto PR not required because: not buggy in crypto
  • framework PR not required
  • 3.6 PR not required because: not present in 3.6
  • tests provided

We were accidentally running the check in TF-PSA-Crypto instead of in Mbed TLS.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
@gilles-peskine-arm gilles-peskine-arm added needs-review Every commit must be reviewed by at least two team members, needs-ci Needs to pass CI tests needs-reviewer This PR needs someone to pick it up for review component-test Test framework and CI scripts priority-high High priority - will be reviewed soon size-xs Estimated task size: extra small (a few hours at most) labels Feb 12, 2026

@yanesca yanesca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-project-automation github-project-automation Bot moved this from In Development to Has Approval in Roadmap pull requests (new board) Feb 12, 2026
@gilles-peskine-arm gilles-peskine-arm added this pull request to the merge queue Feb 12, 2026
Merged via the queue into Mbed-TLS:development with commit 4602f36 Feb 12, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Has Approval to Done in Roadmap pull requests (new board) Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug component-test Test framework and CI scripts needs-ci Needs to pass CI tests needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review priority-high High priority - will be reviewed soon size-xs Estimated task size: extra small (a few hours at most)

Development

Successfully merging this pull request may close these issues.

3 participants