Skip to content

Minor updates to policy config and other readme changes#130

Merged
sureshcsdp merged 4 commits intomainfrom
policy_doc_updates
Apr 7, 2026
Merged

Minor updates to policy config and other readme changes#130
sureshcsdp merged 4 commits intomainfrom
policy_doc_updates

Conversation

@sureshcsdp
Copy link
Copy Markdown
Collaborator

No description provided.

if output in ("json", "csv") and not output_file:
raise click.UsageError(f"--output-file is required when using --output {output}")

# Load policy config early — provider/regions may be resolved from it
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we do some loggign which shows where this picks the config from ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done, just added and pushed

@javvaji-devops
Copy link
Copy Markdown
Collaborator

@sureshcsdp can we release next minor version as policy changes just kickedin?

@javvaji-devops
Copy link
Copy Markdown
Collaborator

LGTM 👍

@sureshcsdp sureshcsdp merged commit 679286c into main Apr 7, 2026
19 checks passed
@sureshcsdp sureshcsdp deleted the policy_doc_updates branch April 7, 2026 19: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.

2 participants