Skip to content
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

Checkmarx/CheckmarxOne: update include/exclude stash file patterns #4358

Merged
merged 3 commits into from
May 15, 2023
Merged

Checkmarx/CheckmarxOne: update include/exclude stash file patterns #4358

merged 3 commits into from
May 15, 2023

Conversation

thtri
Copy link
Member

@thtri thtri commented May 11, 2023

Changes

This PR contains two changes:

  • for the existing CxSAST step: fix the missing include/exclude stash file patterns in pipelineStashFilesBeforeBuild step

  • for the new CxOne step: add the stash file patterns for both pipelineStashFilesBeforeBuild and pipelineStashFilesAfterBuild steps

  • Tests

  • Documentation

@thtri thtri requested a review from a team as a code owner May 11, 2023 17:00
@thtri thtri requested a review from hubadr May 11, 2023 17:01
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@thtri
Copy link
Member Author

thtri commented May 15, 2023

/it-go

@thtri thtri merged commit 659cf9f into SAP:master May 15, 2023
@thtri thtri deleted the cxStash branch September 19, 2023 18:51
maxatsap pushed a commit to maxatsap/jenkins-library that referenced this pull request Jul 23, 2024
@thtri @maxatsap
Checkmarx/CheckmarxOne: update include/exclude stash file patterns (S…
2ecd2d5 
…AP#4358)

* feat(checkmarxOne): add default file patterns for stash

* fix(checkmarx): add missing stash file patterns

* fix(checkmarx-checmarxOne): support TypeScript (issue SAP#3073)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sumeetpatil sumeetpatil sumeetpatil approved these changes

@hubadr hubadr Awaiting requested review from hubadr

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
@thtri @sumeetpatil