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

Move scripts used by all-core.sh to the framework #9788

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

Move scripts used by all-core.sh to the framework #9788

wants to merge 4 commits into from
+3 −303

Conversation

eleuzi01
Copy link
Contributor

@eleuzi01 eleuzi01 commented Nov 20, 2024 •
edited
Loading

Description

Resolves Mbed-TLS/mbedtls-framework#74

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

Notes for the submitter

Please refer to the contributing guidelines, especially the
checklist for PR contributors.

Help make review efficient:

  • Multiple simple commits
    • please structure your PR into a series of small commits, each of which does one thing
  • Avoid force-push
    • please do not force-push to update your PR - just add new commit(s)
  • See our Guidelines for Contributors for more details about the review process.

@eleuzi01 eleuzi01 added enhancement needs-backports Backports are missing or are pending review and approval. needs-ci Needs to pass CI tests size-s Estimated task size: small (~2d) priority-high High priority - will be reviewed soon labels Nov 20, 2024
@eleuzi01 eleuzi01 force-pushed the issue-74-fw branch 2 times, most recently from 0241f74 to 3200cb7 Compare November 20, 2024 15:25
This was referenced Nov 20, 2024
Open
Open
@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 Nov 21, 2024
@eleuzi01 eleuzi01 added the needs-ci Needs to pass CI tests label Nov 25, 2024
@eleuzi01 eleuzi01 removed the needs-ci Needs to pass CI tests label Nov 27, 2024
@eleuzi01
Move files out of Mbed TLS
43a0534 
Signed-off-by: Elena Uziunaite <[email protected]>
@eleuzi01
Adapt paths for output_env.sh
f569ad3 
Signed-off-by: Elena Uziunaite <[email protected]>
@eleuzi01
Adapt paths for scripts/quiet
a4c18da 
Signed-off-by: Elena Uziunaite <[email protected]>
@eleuzi01
Update submodule
fcb7e3b 
Signed-off-by: Elena Uziunaite <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidhorstmann-arm davidhorstmann-arm Awaiting requested review from davidhorstmann-arm

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

Assignees
No one assigned
Labels
enhancement needs-backports Backports are missing or are pending review and approval. 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-s Estimated task size: small (~2d)
Projects
Roadmap pull requests (new board)
Status: In Development
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Move scripts used by all-core.sh to the framework
1 participant
@eleuzi01