Skip to content

feat: add --models, --regions, and --verbose support #77

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

Merged

Conversation

Priyanka-Microsoft
Copy link
Contributor

This pull request enhances the quota_check documentation and script by introducing verbose logging, improving parameter handling, and refining the output format. These changes aim to make the quota-checking process more user-friendly and traceable.

Documentation Updates:

  • Updated docs/quota_check.md to include a new --verbose option for detailed logging, along with examples demonstrating its usage. Clarified parameter handling for models and regions. [1] [2]

Script Enhancements:

  1. Verbose Logging:

    • Added a --verbose flag to enable detailed logging for debugging purposes. Introduced a log_verbose function to conditionally log messages based on the flag. [1] [2] [3] [4] [5]
  2. Improved Parameter Handling:

    • Replaced positional arguments with named parameters (--models, --regions, --verbose) for better readability and flexibility. Defaults are used if parameters are not provided. [1] [2]
  3. Output Refinements:

    • Adjusted output table formatting for better alignment and readability. [1] [2]

These changes collectively enhance the usability and maintainability of the quota-checking process.

@Roopan-Microsoft Roopan-Microsoft self-requested a review April 17, 2025 14:12
@Roopan-Microsoft Roopan-Microsoft merged commit ba65c5f into main Apr 22, 2025
1 check passed
Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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