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

feat(parser): add entry point for cli #3

Merged
merged 2 commits into from
Nov 10, 2021
Merged

feat(parser): add entry point for cli #3

merged 2 commits into from
Nov 10, 2021

Conversation

saranyailla
Copy link
Member

Issue #, if available:

Description of changes:
Initial PR with directory structure and entry point code to build the cli tool using argparse. This also includes setup script to build and install the CLI tool.

Why is this change necessary:

How was this change tested:
coverage run --source=greengrassTools -m pytest -v tests && coverage report --show-missing --fail-under=70

Any additional information or context required to review the change:

  • GitHub workflow
    • Added testing and uploading coverage report to GitHub workflow.
    • Added check for minimum coverage value of 70.
    • Using python 3.6 instead of 3.9 for compatibility check with minimum version support.

Checklist:

  • Updated the README if applicable
  • Updated or added new unit tests
  • Updated or added new integration tests
  • Updated or added new end-to-end tests
  • If your code makes a remote network call, it was tested with a proxy
  • You confirm that the change is backwards compatible

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@saranyailla saranyailla merged commit 50b815b into main Nov 10, 2021
@saranyailla saranyailla deleted the cli-internal branch November 10, 2021 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants