Skip to content

[preview] apply enhanced analysis rules - trial 2#38691

Closed
archerzz wants to merge 2 commits intoAzure:mainfrom
archerzz:enhanced-analysis/trial-2
Closed

[preview] apply enhanced analysis rules - trial 2#38691
archerzz wants to merge 2 commits intoAzure:mainfrom
archerzz:enhanced-analysis/trial-2

Conversation

@archerzz
Copy link
Copy Markdown
Member

  • update the client sdk analyzer to alpha build
  • add globla config to suppress enhanced analysis rules
  • modify ci steps to support suppression by a property
  • applying the property to already impacted sdk projects for backward compatibility

preview for Azure/azure-sdk-tools#6916

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

- update the client sdk analyzer to alpha build
- add globla config to suppress enhanced analysis rules
- modify ci steps to support suppression by a property
- applying the property to already impacted sdk projects for backward compatibility
@archerzz archerzz force-pushed the enhanced-analysis/trial-2 branch from 946e35a to e03cb92 Compare September 14, 2023 02:35
@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

API changes are not detected in this pull request.

@archerzz archerzz closed this Sep 14, 2023
@scottaddie scottaddie added Monitor Monitor, Monitor Ingestion, Monitor Query and removed Monitor - Query labels Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants