Skip to content

Split StaticAnalysis exception files into separate files per module #6298

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
Merged
merged 5 commits into from
May 30, 2018
Merged

Split StaticAnalysis exception files into separate files per module #6298

merged 5 commits into from
May 30, 2018

Conversation

cormacpayne
Copy link
Member

@cormacpayne cormacpayne commented May 25, 2018 •
edited
Loading

Description

Fix for issue #6112

Split .csv files used for StaticAnalysis exceptions into separate files for each module

Checklist

@maddieclayton
Copy link
Contributor

LGTM once documentation is updated.

markcowl
markcowl previously approved these changes May 26, 2018
View reviewed changes
Copy link
Member

@markcowl markcowl left a comment

Choose a reason for hiding this comment

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

LGTM, please update docs

@markcowl
Copy link
Member

on demand run here: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/powershell-demand/886/

@maddieclayton maddieclayton mentioned this pull request May 29, 2018
Merged
8 tasks
@cormacpayne
Copy link
Member Author

On-demand: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/powershell-demand/893/

maddieclayton
maddieclayton previously approved these changes May 29, 2018
View reviewed changes
@maddieclayton maddieclayton changed the base branch from preview to release-6.2.0 May 30, 2018 00:10
@cormacpayne
Copy link
Member Author

On-demand: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/powershell-demand/901/

@maddieclayton maddieclayton merged commit bd69cc2 into Azure:release-6.2.0 May 30, 2018
@cormacpayne cormacpayne deleted the exceptions-refactor branch September 12, 2018 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maddieclayton maddieclayton maddieclayton approved these changes

@markcowl markcowl markcowl left review comments

Assignees
No one assigned
Labels
Waiting for On Demand
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cormacpayne @maddieclayton @markcowl @azuresdkci