Skip to content

Removed feature flag new-sast-scan-result-enabled(AST-99956) #1197

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.

Sign up for GitHub

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

Jump to bottom
Merged
cx-anjali-deore merged 3 commits into from
Jun 30, 2025
Merged

Removed feature flag new-sast-scan-result-enabled(AST-99956) #1197

cx-anjali-deore merged 3 commits into from
Jun 30, 2025

Conversation

@cx-anjali-deore
Copy link
Contributor

@cx-anjali-deore cx-anjali-deore commented Jun 23, 2025 •
edited
Loading

By submitting this pull request, you agree to the terms within the Checkmarx Code of Conduct. Please review the contributing guidelines for guidance on creating high-quality pull requests.

Description

*Removing the FF NEW-SAST-SCAN-RESULT-ENABLED as its enabled for all the customers in production. *
Removing the logic when flag is off

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Related Issues

Link any related issues or tickets.

Checklist

  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules
  • 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

Screenshots (if applicable)

Add screenshots to help explain your changes.

Additional Notes

Add any other relevant information.

@cx-anjali-deore cx-anjali-deore changed the title removed FF NEW_SAST_SCAN_REPORT_ENABLED(AST-99956) removed feature falg new-sast-scan-result-enabled(AST-99956) Jun 23, 2025
@cx-anjali-deore cx-anjali-deore changed the title removed feature falg new-sast-scan-result-enabled(AST-99956) Removed feature flag new-sast-scan-result-enabled(AST-99956) Jun 23, 2025
@github-actions
Copy link

github-actions bot commented Jun 23, 2025 •
edited
Loading

Logo
Checkmarx One – Scan Summary & Details – f8103d75-6c81-4db5-8597-4735abe5c20d

Great job, no security vulnerabilities found in this Pull Request

cx-anurag-dalke
cx-anurag-dalke approved these changes Jun 26, 2025
View reviewed changes
Copy link
Collaborator

@cx-anurag-dalke cx-anurag-dalke left a comment

Choose a reason for hiding this comment

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

ok

@cx-anjali-deore cx-anjali-deore force-pushed the other/FF-NewScanReport branch from 525bcec to 71d20a1 Compare June 27, 2025 10:30
@cx-anjali-deore cx-anjali-deore merged commit 0af147c into main Jun 30, 2025
16 of 19 checks passed
@cx-anjali-deore cx-anjali-deore deleted the other/FF-NewScanReport branch June 30, 2025 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cx-anurag-dalke cx-anurag-dalke cx-anurag-dalke approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cx-anjali-deore @cx-anurag-dalke