-
Notifications
You must be signed in to change notification settings - Fork 607
Changed the documents link from Synopsys to Blackduck #5384
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.
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
base: master
Are you sure you want to change the base?
Conversation
/it-go |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine beside the product name spelling. Do you have any link to more information about the product renaming? This would be good to have here on the PR for future reference.
This step executes [Synopsys Detect](https://community.synopsys.com/s/document-item?bundleId=integrations-detect&topicId=introduction.html&_LANG=enus) scans. | ||
Synopsys Detect command line utlity can be used to run various scans including BlackDuck and Polaris scans. This step allows users to run BlackDuck scans by default. | ||
This step executes [Blackduck Detect](https://documentation.blackduck.com/bundle/detect/page/introduction.html) scans. | ||
Blackduck Detect command line utlity can be used to run various scans including BlackDuck scans. This step allows users to run BlackDuck scans by default. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the product name written with a capital or lower-case d
? Please be consistent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CCFenner Thank you for highlighting, I have changed this to BlackDuck to be consistent. I will also take care in future.
Synopsys sold Software security Business to Private equity firm last year and rebranded whole business as Black Duck. Here is the public notice for the same: |
|
/it-go |
Description
This pull request updates the document references for the detectExecuteScan step. Since Synopsys Detect has transitioned to Black Duck domain, the previous links had become invalid. This update includes the correct set of links required for the Black Duck scan.
Checklist