Skip to content

Conversation

@cx-itay-paz
Copy link
Contributor

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Support custom states in the cli included --all flag (deleted states).

References

https://checkmarx.atlassian.net/browse/AST-80216

Testing

I tested whether the get-states command correctly retrieves and displays custom states, ensuring that the --all flag includes deleted states and that no extra spaces or newline characters appear in the output.

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • I have updated the CLI help for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@github-actions
Copy link

github-actions bot commented Feb 2, 2025

Logo
Checkmarx One – Scan Summary & Details6dec86de-3c9c-43d4-a70a-4eb7d5275d8c

Great job, no security vulnerabilities found in this Pull Request

Copy link
Collaborator

@cx-ben-alvo cx-ben-alvo left a comment

Choose a reason for hiding this comment

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

Great job!

@cx-itay-paz cx-itay-paz merged commit 2a8e0c6 into main Feb 4, 2025
9 of 10 checks passed
@cx-itay-paz cx-itay-paz deleted the add-a-new-flag-to-get-all-custom-states branch February 4, 2025 14:23
cx-anjali-deore pushed a commit that referenced this pull request Jul 3, 2025
…tom-states

Suppot custom states sub command with -all flag (AST-80216)
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