Skip to content
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

Bump Microsoft.Private.Intellisense package version #91285

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

carlossanlop
Copy link
Member

Generated it via the dotnet-api-docs pipeline: https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=382697&view=logs&j=fd490c07-0b22-5182-fac9-6d67fe1e939b&t=11e7ea89-affe-5194-cdc6-0171c3394706

This is not the latest version of the package but adds docs for more than half of the APIs added in .NET 8.

Will backport.

@carlossanlop carlossanlop added documentation Documentation bug or enhancement, does not impact product or test code area-Infrastructure-libraries labels Aug 29, 2023
@carlossanlop carlossanlop self-assigned this Aug 29, 2023
@ghost
Copy link

ghost commented Aug 29, 2023

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Generated it via the dotnet-api-docs pipeline: https://apidrop.visualstudio.com/Content%20CI/_build/results?buildId=382697&view=logs&j=fd490c07-0b22-5182-fac9-6d67fe1e939b&t=11e7ea89-affe-5194-cdc6-0171c3394706

This is not the latest version of the package but adds docs for more than half of the APIs added in .NET 8.

Will backport.

Author: carlossanlop
Assignees: carlossanlop
Labels:

documentation, area-Infrastructure-libraries

Milestone: -

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

Approving but CI is failing

@carlossanlop
Copy link
Member Author

The problem was that we're still pushing to dotnet7-transport. I manually pushed the private package to the dotnet8-transport.
Closing and reopening to trigger the CI again.

@ViktorHofer
Copy link
Member

ViktorHofer commented Aug 30, 2023

Now source build tells you that you are introducing a new prebuilt. You need to add the package to https://github.com/dotnet/source-build-reference-packages. cc @mthalman @MichaelSimons

EDIT:
You can do that via the following command: generate.cmd/sh -p Microsoft.Private.Intellisense,8.0.0-preview-20230828.1 -t text

@mthalman
Copy link
Member

Be aware of this: #77004

@carlossanlop
Copy link
Member Author

carlossanlop commented Sep 1, 2023

Thanks @ViktorHofer for the instructions, and thanks @mthalman @MichaelSimons for the warning (#77004).

I want to reduce the burden of having to manually add a new package version to source-build-reference-packages. So I would like to do this update process only two times:

  1. I'd like to merge this PR (after merging the other repo's PR Manually update Microsoft.Private.Intellisense to 8.0.0-preview-20230828.1 source-build-reference-packages#774)) to make sure everything works as expected and nothing breaks.
  2. I'll submit only one more PR doing this same process, with the final package with docs for .NET 8.

Does this sound good to everyone?

@carlossanlop
Copy link
Member Author

I would also like to learn how to automate this, or completely get rid of the necessity of manually updating the package in source-build-reference-packages, but I don't know what's needed to achieve this. Who can guide me through this?

@ViktorHofer
Copy link
Member

but I don't know what's needed to achieve this. Who can guide me through this?

One thing: make the XML documentation files source of truth the repositories themselves. As long as we rely on an external repository supplying a nuget package with the documentation XML files, we will always have prebuilds.

@carlossanlop
Copy link
Member Author

We merged dotnet/source-build-reference-packages#774 so it should be safe to merge this now.

@carlossanlop carlossanlop merged commit 4f8badb into main Sep 1, 2023
176 of 180 checks passed
@carlossanlop carlossanlop deleted the BumpMicrosoftPrivateIntellisense branch September 1, 2023 21:37
@carlossanlop
Copy link
Member Author

/backport to release/8.0

@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2023

Started backporting to release/8.0: https://github.com/dotnet/runtime/actions/runs/6054208982

@ViktorHofer
Copy link
Member

ViktorHofer commented Sep 2, 2023

We merged dotnet/source-build-reference-packages#774 so it should be safe to merge this now.

No. This nukes the build. You need to wait for a dependency update from source-build-reference-packages into runtime. Please never merge a PR in with a failing source build leg. I will take care of fixing the build.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure-libraries documentation Documentation bug or enhancement, does not impact product or test code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants