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

Improvements to the Polaris CLI #30

Merged
merged 6 commits into from
Jul 31, 2024
Merged

Commits on Jul 31, 2024

  1. SNOW-1527866: [3/x] Improvements to the Polaris CLI (apache#21)

    <!-- Please describe your change here and remove this comment -->
    This PR covers various improvements to the [recently
    merged](https://github.com/snowflakedb/managed-pull/7) Polaris
    CLI. Changes here include:
    
    * Added the `namespaces` command for creating, listing, and dropping
    namespaces with the CLI
    * Refactored error handling to reduce the proliferation of string
    literals
    * Added support for the `FILE` storage type
    * Added end-to-end regression tests for the CLI
    * Various usability and bug fixes
    * Support for the `CLIENT_ID` and `CLIENT_SECRET` environment variables
    * CLI documentation
    
    ## Pre-review checklist
    - [ ] I attest that this change meets the bar for low risk without
    security requirements as defined in the [Accelerated Risk Assessment
    Criteria](https://developer-handbook.m1.us-west-2.aws.app.snowflake.com/docs/reference/security-review/accelerated-risk-assessment/#eligibility)
    and I have taken the [Risk Assessment Training in
    Workday](https://wd5.myworkday.com/snowflake/learning/course/6c613806284a1001f111fedf3e4e0000).
    - Checking this checkbox is mandatory if using the [Accelerated Risk
    Assessment](https://developer-handbook.m1.us-west-2.aws.app.snowflake.com/docs/reference/security-review/accelerated-risk-assessment/)
    to risk assess the changes in this Pull Request.
    - If this change does not meet the bar for low risk without security
    requirements (as confirmed by the peer reviewers of this pull request)
    then a [formal Risk
    Assessment](https://developer-handbook.m1.us-west-2.aws.app.snowflake.com/docs/reference/security-review/risk-assessment/)
    must be completed. Please note that a formal Risk Assessment will
    require you to spend extra time performing a security review for this
    change. Please account for this extra time earlier rather than later to
    avoid unnecessary delays in the release process.
    - [ ] This change has code coverage for the new code added
    eric-maynard committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    cfc71e8 View commit details
    Browse the repository at this point in the history
  2. license headers

    eric-maynard committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    281ee95 View commit details
    Browse the repository at this point in the history
  3. more headers

    eric-maynard committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    04351e2 View commit details
    Browse the repository at this point in the history
  4. revert?

    eric-maynard committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    183ef85 View commit details
    Browse the repository at this point in the history
  5. changes per comments

    eric-maynard committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    bd4e462 View commit details
    Browse the repository at this point in the history
  6. rerun tests

    eric-maynard committed Jul 31, 2024
    Configuration menu
    Copy the full SHA
    e891231 View commit details
    Browse the repository at this point in the history