-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
[3.6] Move "easy" basic checks scripts to the framework #9826
[3.6] Move "easy" basic checks scripts to the framework #9826
Conversation
4211565
to
586362c
Compare
a8f5b9b
to
0c7ce15
Compare
ChangeLog.d/00README.md
Outdated
@@ -86,6 +86,6 @@ See [existing entries](../ChangeLog) for examples. | |||
|
|||
## How `ChangeLog` is updated | |||
|
|||
Run [`../scripts/assemble_changelog.py`](../scripts/assemble_changelog.py) | |||
Run [`../framework/scripts/assemble_changelog.py`](../scripts/assemble_changelog.py) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run [`../framework/scripts/assemble_changelog.py`](../scripts/assemble_changelog.py) | |
Run [`../framework/scripts/assemble_changelog.py`](../framework/scripts/assemble_changelog.py) |
0c7ce15
to
c34b767
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The following files are moved to the framework repo (deleted here): scripts/assemble_changelog.py tests/scripts/check-doxy-blocks.pl tests/scripts/check-python-files.sh tests/scripts/doxygen.sh scripts/apidoc_full.sh tests/scripts/recursion.pl Signed-off-by: Valerio Setti <[email protected]>
Signed-off-by: Valerio Setti <[email protected]>
c34b767
to
a70dcee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Move "easy" basic checks scripts to the framework PR validated by the CI of Mbed-TLS/mbedtls#9825 and Mbed-TLS/mbedtls#9826. The files moved are not used yet in TF-PSA-Crypto thus no impact there. Merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
Signed-off-by: Valerio Setti <[email protected]>
29a4785
a70dcee
to
29a4785
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Resolves Mbed-TLS/mbedtls-framework#69 on the main repo in the
mbedtls-3.6
branchPR checklist