Skip to content

Conversation

@cx-ben-alvo
Copy link
Collaborator

@cx-ben-alvo cx-ben-alvo commented Nov 20, 2024

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

Added a method that write a single value to .checkmarx.yml

References

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

Testing

Added unit tests

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

@OrShamirCM OrShamirCM requested a review from a team November 20, 2024 14:54
@github-actions github-actions bot added the bug Something isn't working label Nov 20, 2024
@github-actions
Copy link

github-actions bot commented Nov 20, 2024

Logo
Checkmarx One – Scan Summary & Details5b3adf96-c533-4e13-ac3f-ab81026d7b68

No New Or Fixed Issues Found

OrShamirCM
OrShamirCM previously approved these changes Nov 20, 2024
@cx-ben-alvo cx-ben-alvo changed the title Bug/benalvo/set asca port in config Fix ASCA Port Overwrite Issue in .checkmarx.yml (AST-75444) Nov 21, 2024
OrShamirCM
OrShamirCM previously approved these changes Nov 21, 2024
OrShamirCM
OrShamirCM previously approved these changes Nov 24, 2024
@cx-ben-alvo cx-ben-alvo merged commit 10947ef into main Nov 24, 2024
8 checks passed
@cx-ben-alvo cx-ben-alvo deleted the bug/benalvo/set-asca-port-in-config branch November 24, 2024 12:48
cx-anjali-deore pushed a commit that referenced this pull request Jul 3, 2025
* added write single param to config file function

* refactor

* fix coverage

* fix linter

* fix linter

* add locking mechanism when writing to the .checkmarx.yml

* chenge getConfigFilePath to be private

* refactor

* fix compilation errors

* fix lint

* fix lint

* add unittests

* add unittests

* Change coverage

* fix lint

* refactor

* fix unitests

* Resolve conversations

---------

Co-authored-by: AlvoBen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants