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.
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
[release/7.0] Mark API as shipped #46042
Conversation
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.
Stopped after a dozen or so files because this is difficult to view w/o the corresponding Unshipped files.
@@ -1,4 +1,5 @@ | |||
#nullable enable |
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.
Why aren't the Unshipped.txt files all being emptied in this 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.
I'm not sure, I just ran ./eng/scripts/mark-shipped.cmd
and committed everything (except the submodule change). Can dig in more today.
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.
I think in this case it just moved lines around?
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.
Yeah, just moving lines around because the script sorts things alphabetically.
Just curious: What was your process here❔ For example, did you run eng\scripts\mark-shipped.cmd then |
Yes |
Huge number of errors here, we may need to update the |
Scratch that, I think the errors are all because the script did a weird thing with spacing (when moving lines to *.shipped.txt, it put them all on the same line). Will see if I can figure out why, rework the script, then re-run it. |
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.
Looked through ~20% of the files and didn't see anything to worry about.
Only failures are in Code Check about modifying baseline fails - merging |
Move the APIs we shipped in 7 from
Unshipped.txt
toShipped.txt