[PR] CLI: Fix catalog command [polaris]

["MonkeyCanCode (via GitHub)" <gi...@apache.org>]

MonkeyCanCode opened a new pull request, #4047:
URL: https://github.com/apache/polaris/pull/4047

   <!--
   📝 Describe what changes you're proposing, especially breaking or user-facing changes. 
   📖 See https://github.com/apache/polaris/blob/main/CONTRIBUTING.md for more.
   -->
   
   This PR fixed two things fro catalogs CLI:
   1. Missing `'` for `SIGV4`
   2. For second comparison with catalog_service_identity_type, we are not compare with value for enum
   
   ## Checklist
   - [ ] 🛡️ Don't disclose security issues! (contact security@apache.org)
   - [ ] 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #
   - [ ] 🧪 Added/updated tests with good coverage, or manually tested (and explained how)
   - [ ] 💡 Added comments for complex logic
   - [ ] 🧾 Updated `CHANGELOG.md` (if needed)
   - [ ] 📚 Updated documentation in `site/content/in-dev/unreleased` (if needed)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] CLI: Fix catalog command [polaris]

["MonkeyCanCode (via GitHub)" <gi...@apache.org>]

MonkeyCanCode merged PR #4047:
URL: https://github.com/apache/polaris/pull/4047


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org