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

[development] Add components to components-build-system.sh #10010

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

[development] Add components to components-build-system.sh #10010

wants to merge 2 commits into from

Conversation

valeriosetti
Copy link
Contributor

@valeriosetti valeriosetti commented Feb 25, 2025 •
edited by ronald-cron-arm
Loading

Description

Resolves Mbed-TLS/TF-PSA-Crypto#137

PR checklist

@valeriosetti valeriosetti 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 size-s Estimated task size: small (~2d) priority-high High priority - will be reviewed soon labels Feb 25, 2025
@valeriosetti valeriosetti self-assigned this Feb 25, 2025
@valeriosetti valeriosetti mentioned this pull request Feb 25, 2025
Merged
3 tasks
Harry-Ramsey
Harry-Ramsey previously approved these changes Feb 25, 2025
View reviewed changes
@valeriosetti valeriosetti removed the needs-ci Needs to pass CI tests label Feb 26, 2025
@valeriosetti valeriosetti force-pushed the issue137-development branch from da1898a to e4322c0 Compare March 4, 2025 13:38
@valeriosetti valeriosetti force-pushed the issue137-development branch from e4322c0 to e02c757 Compare March 5, 2025 10:42
@valeriosetti valeriosetti removed the needs-reviewer This PR needs someone to pick it up for review label Mar 5, 2025
@valeriosetti valeriosetti force-pushed the issue137-development branch from e02c757 to 5468611 Compare March 6, 2025 10:03
ronald-cron-arm
ronald-cron-arm previously approved these changes Mar 6, 2025
View reviewed changes
Copy link
Contributor

@ronald-cron-arm ronald-cron-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, thanks.

Harry-Ramsey
Harry-Ramsey previously approved these changes Mar 6, 2025
View reviewed changes
@valeriosetti
tests: move component_test_tf_psa_crypto_cmake_as_package to tf-psa-c…
6408113 
…rypto

Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
@valeriosetti
tf-psa-crypto: update reference
5ec134f 
Signed-off-by: Valerio Setti <valerio.setti@nordicsemi.no>
@valeriosetti valeriosetti force-pushed the issue137-development branch from 5468611 to 5ec134f Compare March 10, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ronald-cron-arm ronald-cron-arm ronald-cron-arm approved these changes

@Harry-Ramsey Harry-Ramsey Harry-Ramsey approved these changes

Assignees

@valeriosetti valeriosetti

Labels
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 pull requests (new board)
Status: Has Approval
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add components to components-build-system.sh
3 participants
@valeriosetti @ronald-cron-arm @Harry-Ramsey