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: Typer CLI implementation with cognito device authorization flow #88

Closed
wants to merge 23 commits into from

Conversation

ar-aws
Copy link

@ar-aws ar-aws commented Aug 27, 2024

Reason for this change

Builders want more flexible methods of interacting with the SBT Control plane and a CLI client provides them an additional way to leverage the platform.

Description of changes

  • Implemented CLI and published as an PyPI package sbt-aws-cli.
  • Chose Typer framework due to its enhanced user interface and ease of development.
  • Set up a GitHub Actions workflow to automate the package build and publish process, ensuring that any updates to the CLI are automatically deployed to PyPI upon changes to the pyproject.toml file.
  • Added documentation in README for the CLI
  • Created cli-auth.ts which houses resources for OAuth 2.0 device authorization flow to ensure secure authentication for CLI users

Description of how you validated changes

  • Manually tested the CLI commands to ensure they behave as expected and interact correctly with the underlying services.
  • Validated the OAuth 2.0 device authorization flow to ensure it handles authentication securely and appropriately.
  • Ran the GitHub Actions workflow to confirm that the build and publish process works correctly
  • Added bash script to test the new CLI

Checklist

  • My code adheres to the CONTRIBUTING GUIDE
  • I have updated the relevant documentation (if applicable).

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

docs/public/README.md Outdated Show resolved Hide resolved
@suhussai suhussai force-pushed the command-line-interface branch from 03c89a3 to b622b6d Compare September 6, 2024 17:45
@suhussai
Copy link
Contributor

Replaced by PR #93 which is similar but also contains a fix for the failing build job.

@suhussai suhussai closed this Sep 10, 2024
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.

2 participants