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.

Sign up for GitHub

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

Jump to bottom

Add name checks #97

Open
wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Add name checks #97

wants to merge 2 commits into from

Conversation

gabor-mezei-arm
Copy link

Adapt tests/scripts/check_names.py to work for both mbedtls and psa-crypto.
Add component_check_names all.sh component.

Resolve #52

Depends on #58, Mbed-TLS/mbedtls#8651

@gabor-mezei-arm
Add check_names.py to psa_crypto.py
9f3ad8f 
Signed-off-by: Gabor Mezei <gabor.mezei@arm.com>
@gabor-mezei-arm
Add check_names all.sh component
ecd81f5 
Signed-off-by: Gabor Mezei <gabor.mezei@arm.com>
@gabor-mezei-arm gabor-mezei-arm added enhancement New feature or request size-s Estimated task size: small (~2d) needs-review Every commit must be reviewed by at least two team members priority-high High priority - will be reviewed soon labels Dec 21, 2023
@gabor-mezei-arm gabor-mezei-arm self-assigned this Dec 21, 2023
@gabor-mezei-arm gabor-mezei-arm linked an issue Jan 5, 2024 that may be closed by this pull request
Add name checks #52
Open
2 tasks
tom-daubney-arm
tom-daubney-arm approved these changes Jan 23, 2024
View reviewed changes
Copy link

@tom-daubney-arm tom-daubney-arm left a comment

Choose a reason for hiding this comment

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

LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tom-daubney-arm tom-daubney-arm tom-daubney-arm approved these changes

@ronald-cron-arm ronald-cron-arm Awaiting requested review from ronald-cron-arm

At least 2 approving reviews are required to merge this pull request.

Assignees

@gabor-mezei-arm gabor-mezei-arm

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
Roadmap Board for Mbed TLS
Status: To Do
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add name checks
2 participants
@gabor-mezei-arm @tom-daubney-arm