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 most of min_requirements.py to the framework #86

Open
mpg opened this issue Dec 2, 2024 · 0 comments
Open

Move most of min_requirements.py to the framework #86

mpg opened this issue Dec 2, 2024 · 0 comments
Labels
size-s Estimated task size: small (~2d)

Comments

@mpg
Copy link
Contributor

mpg commented Dec 2, 2024

We want to keep scripts/min_requirements.py in each branch (3.6 and development), as that's the entry point that the CI is calling. But we want to move most of its code to the framework, as a module that can be imported. So, the end result would probably have scripts/min_requirements.py in each branch looking like

import framework_scripts_path # pylint: disable=unused-import
from mbedtls_framework import min_requirements

min_requirements.main()

and then the current body of scripts/min_requirements.py would be moved to the framework repo, in scripts/mbedtls_framework (as a non-executable file).

In terms of history, we probably want to first completely move min_requirements.py to the framework (so that its full history is present there) and then add back the new stub in 3.6 and development.

@mpg mpg added the size-s Estimated task size: small (~2d) label Dec 2, 2024
@mpg mpg moved this to Framework 2/3 (script for the CI) in Mbed TLS Epics Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size-s Estimated task size: small (~2d)
Projects
Status: Framework 2/3 (script for the CI)
Development

No branches or pull requests

1 participant