Skip to content

[xamarin-analysis] Add rule #6 and updated md file - #3088

Merged
VincentDondain merged 1 commit into
dotnet:masterfrom
VincentDondain:xia-0006-update
Dec 12, 2017
Merged

[xamarin-analysis] Add rule #6 and updated md file#3088
VincentDondain merged 1 commit into
dotnet:masterfrom
VincentDondain:xia-0006-update

Conversation

@VincentDondain

Copy link
Copy Markdown
Contributor
  • Add XIA0006: HttpClientAvoidManaged.
  • Add documentation on how the rules work and how to activate them.
    Also mention that they need to be ran on each active configuration.
  • Bump maccore to include XIA 0006.

- Add XIA0006: HttpClientAvoidManaged.
- Add documentation on how the rules work and how to activate them.
  Also mention that they need to be ran on each active configuration.
- Bump maccore to include XIA 0006.
@VincentDondain VincentDondain added the do-not-merge Do not merge this pull request label Dec 7, 2017
@VincentDondain

Copy link
Copy Markdown
Contributor Author

dont-merge because the bots are currently down (:

@spouliot spouliot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This also needs to be added to 11.7 release notes in the wiki

@VincentDondain

Copy link
Copy Markdown
Contributor Author

@spouliot done here https://github.com/xamarin/xamarin-macios/wiki/xamarin.ios_11.7#xamarinanalysis

@monojenkins

Copy link
Copy Markdown
Contributor

Build success

@VincentDondain VincentDondain removed the do-not-merge Do not merge this pull request label Dec 12, 2017
@VincentDondain

Copy link
Copy Markdown
Contributor Author

@VincentDondain
VincentDondain merged commit 79e1d36 into dotnet:master Dec 12, 2017
@VincentDondain
VincentDondain deleted the xia-0006-update branch December 12, 2017 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants