Skip to content
This repository has been archived by the owner on May 5, 2023. It is now read-only.

fix-security-issue-and-add-credscan #5220

Merged
merged 2 commits into from Nov 25, 2021

Conversation

qiaozha
Copy link
Member

@qiaozha qiaozha commented Nov 24, 2021

No description provided.

@HarshaNalluru
Copy link
Member

I see they are commented in the testlist.txt, we don't intend to run the tests for them? Is it because they are deprecated/moved out of the -node repo?

@qiaozha
Copy link
Member Author

qiaozha commented Nov 25, 2021

I see they are commented in the testlist.txt, we don't intend to run the tests for them? Is it because they are deprecated/moved out of the -node repo?

I am not sure but from the comment https://github.com/Azure/azure-sdk-for-node/blob/master/test/testlist.txt#L29 in testlist.txt, they are removed because they fail to run those test at that time

@qiaozha qiaozha merged commit d5f8742 into master Nov 25, 2021
@qiaozha qiaozha deleted the fix-security-issue-and-add-credscan branch November 25, 2021 01:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants