Skip to content
This repository was archived by the owner on May 3, 2024. It is now read-only.

Update whitesource configuration#1372

Merged
mukul-seagate11 merged 1 commit intomainfrom
swanand-gadre-update-whitesource-config-1
Dec 22, 2021
Merged

Update whitesource configuration#1372
mukul-seagate11 merged 1 commit intomainfrom
swanand-gadre-update-whitesource-config-1

Conversation

@swanand-gadre
Copy link
Copy Markdown
Contributor

Signed-off-by: Swanand S Gadre swanand.s.gadre@seagate.com

Describe your changes in brief

Making following changes to whitesource configuration

Correcting whitesource.config file and excluding "doc" folder from scan

Changes

  • Why is this change required? What problem does it solve?
  • If proposing a new change then please raise an issue first

whitesource is reporting many vulnerabilities from "doc" folder.
This contents of this folder are not bundled with production.
So whitesource configuration needs to be modified to ignore "doc" folder.

How Has This Been Tested? (Optional)

  • Please describe in detail how you tested your changes.
  • Include details of your testing environment, and the tests you ran to
  • How your change affects other areas of the code, etc.

Its a configuration file change, so testing is not directly applicable.

Screenshots (if appropriate)

Not applicable

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

Not applicable

Signed-off-by: Swanand S Gadre <swanand.s.gadre@seagate.com>
Copy link
Copy Markdown
Contributor

@mukul-seagate11 mukul-seagate11 left a comment

Choose a reason for hiding this comment

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

looks good

@mukul-seagate11 mukul-seagate11 merged commit efa49a2 into main Dec 22, 2021
@mukul-seagate11 mukul-seagate11 deleted the swanand-gadre-update-whitesource-config-1 branch December 22, 2021 12:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants