CI Update: https://dev.azure.com/msft-skilling/Content/_workitems/edit/569695#313
CI Update: https://dev.azure.com/msft-skilling/Content/_workitems/edit/569695#313
Conversation
Build.Reason:Manual by Alma Jenks (BEYONDSOFT CONSULTING INC) Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=601710&view=results source_repo.branch:master source_repo.csvPath:bundlepackages/aspnetcore source_repo.url:https://apidrop.visualstudio.com/binaries/_git/mrefconfig
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit aa613c8:
|
Build.Reason:Manual by Alma Jenks (BEYONDSOFT CONSULTING INC) Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=602079&view=results source_repo.branch:master source_repo.csvPath:bundlepackages/aspnetcore source_repo.url:https://apidrop.visualstudio.com/binaries/_git/mrefconfig
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 95a5aab:
|
wadepickett
left a comment
wadepickett
left a comment
There was a problem hiding this comment.
Approved. Thanks!!!!
I checked over the warnings and compared them to previous major version updates (with preview) such as v10. Looks like they are the same expected cross-link warnings that were ignored before.
|
@mikekistler, you were on the API ref generation review list for ASP.NET Core for v10, so I added you as an additional reviewer to see what you think. The writer who used to review these is gone and and my confidence in reviewing these is not real high yet. |
|
@wadepickett I also spot checked some of these warnings and they appear to be valid. I've kicked off a copilot run to see if it can figure out how to fix any of them. But that probably should not hold up this PR, as I believe that any fixes we make won't affect this doc set ... not until the next release version. |
|
As always @v-alje, thanks for your awesome work! |
3 participants
Build.Reason:Manual by Alma Jenks (BEYONDSOFT CONSULTING INC)
Build.Url:https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=601710&view=results
source_repo.branch:master
source_repo.csvPath:bundlepackages/aspnetcore
source_repo.url:https://apidrop.visualstudio.com/binaries/_git/mrefconfig