-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Disable API check for newly added packages #6563
Conversation
@@ -8,6 +8,7 @@ | |||
<NoWarn>$(NoWarn);CS1591</NoWarn> | |||
<GenerateDocumentationFile>true</GenerateDocumentationFile> | |||
<PackageTags>aspnetcore;aspnetcoremvc;cshtml;razor</PackageTags> | |||
<EnableApiCheck>false</EnableApiCheck> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should have an empty baseline rather then disabling API check. Then when we go to update all of the baselines the empty one will become full
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh right, that's what we decided to do. I totally forgot.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bad @Eilon
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It takes two people to be bad. One to be do something bad, and the other one to sign off on the PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What Ryan said.
dd0bbcd
to
1f40681
Compare
1f40681
to
c56f2c0
Compare
🆙 📅 |
No description provided.