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

New Project generation feature #124

Merged
merged 11 commits into from
Apr 13, 2023
Merged

Conversation

thomas-bc
Copy link
Contributor

Originating Project/Creator
Affected Component
Affected Architectures(s)
Related Issue(s)
Has Unit Tests (y/n)
Builds Without Errors (y/n) y
Unit Tests Pass (y/n)
Documentation Included (y/n) y

Change Description

Adds in a new project cookiecutter template and the associated logic for new --project
Checks that git is installed, generates a minimal amount of project files, then init a git repo and add F' as a git submodule.
Optionally creates a venv and sets it up with requirements.txt.

Rationale

Vastly improves user experience to getting started, enforces a good project structure instead of using Ref/

Testing/Review Recommendations

fprime-util new --project
cd ...
fprime-util new --deployment
cd ...
fprime-util generate
fprime-util build
fprime-gds

Tested with pip install . with no -e.

@thomas-bc thomas-bc added the enhancement New feature or request label Apr 6, 2023
@thomas-bc thomas-bc requested a review from LeStarch April 6, 2023 01:49
@thomas-bc
Copy link
Contributor Author

@LeStarch There's probably a lot to be said about the generated CMakeLists.txt and settings.ini as this methodology is not clearly defined yet. Should we just remove them for the time being?

print(f"[WARNING] Reverted to default branch: {DEFAULT_BRANCH}")

# Install Venv
if "{{cookiecutter.install_venv}}" == "yes":

Check warning

Code scanning / CodeQL

Constant in conditional expression or statement

Testing a constant will always give the same result.
print(f"[WARNING] Reverted to default branch: {DEFAULT_BRANCH}")

# Install Venv
if "{{cookiecutter.install_venv}}" == "yes":

Check warning

Code scanning / CodeQL

Comparison of constants

Comparison of constants; use 'True' or 'False' instead.
@LeStarch LeStarch merged commit e2056e2 into nasa:devel Apr 13, 2023
@thomas-bc thomas-bc deleted the feat/new-project branch October 11, 2023 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants