Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move check_files.py to the framework #84

Merged
merged 14 commits into from
Dec 19, 2024
Merged

Conversation

eleuzi01
Copy link
Contributor

@eleuzi01 eleuzi01 commented Nov 25, 2024

Resolves #72

Development: Mbed-TLS/mbedtls#9800
3.6: Mbed-TLS/mbedtls#9801

@eleuzi01 eleuzi01 added enhancement New feature or request needs-ci Needs to pass CI tests size-s Estimated task size: small (~2d) priority-high High priority - will be reviewed soon labels Nov 25, 2024
@eleuzi01 eleuzi01 force-pushed the issue-72 branch 3 times, most recently from 991c8d6 to 04c3adf Compare November 29, 2024 16:26
@eleuzi01 eleuzi01 force-pushed the issue-72 branch 3 times, most recently from dd342d5 to e984638 Compare December 10, 2024 15:00
@valeriosetti valeriosetti self-requested a review December 11, 2024 12:33
@eleuzi01 eleuzi01 added 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 and removed needs-ci Needs to pass CI tests labels Dec 11, 2024
@eleuzi01 eleuzi01 force-pushed the issue-72 branch 2 times, most recently from f470548 to a85128f Compare December 13, 2024 14:49
Signed-off-by: Elena Uziunaite <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
@minosgalanakis
Copy link
Contributor

minosgalanakis commented Dec 18, 2024

This PR has been locally verfied against TF-PSA crypto repository without any issues.

The current CI failure is a false positive.

This PR has been verified as part of the CI run for 9800

Copy link
Contributor

@minosgalanakis minosgalanakis left a comment

Choose a reason for hiding this comment

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

LGTM

@eleuzi01 eleuzi01 added needs-review Every commit must be reviewed by at least two team members, and removed 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 labels Dec 19, 2024
@minosgalanakis minosgalanakis merged commit d9d14c4 into Mbed-TLS:main Dec 19, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs-review Every commit must be reviewed by at least two team members, priority-high High priority - will be reviewed soon size-s Estimated task size: small (~2d)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move tests/scripts/check_files.py to the framework
4 participants