Skip to content

obsolete PackageSource related types from earlier in 5.8#3698

Merged
heng-liu merged 3 commits intodevfrom
dev-rrelyea-obsoletenewpackagesourceapis
Oct 1, 2020
Merged

obsolete PackageSource related types from earlier in 5.8#3698
heng-liu merged 3 commits intodevfrom
dev-rrelyea-obsoletenewpackagesourceapis

Conversation

@rrelyea
Copy link
Contributor

@rrelyea rrelyea commented Oct 1, 2020

Bug

related to NuGet/Home#10098
Regression: No

Fix

Details: Obsolete a few apis that were added earlier in 5.8, to avoid customers from using them.

Testing/Validation

Tests Added: No
Validation: will validate insertion build to ensure tools/options/package sources still works in codespaces and non-codespaces.

@rrelyea rrelyea requested a review from a team as a code owner October 1, 2020 00:24
@nkolev92
Copy link
Member

nkolev92 commented Oct 1, 2020

@rrelyea You might not want to say "Fixes", because it will auto close the issue.

We probably need another issue for adding the obsolete flag on these APIs and keep an issue as a follow to remove them.

@heng-liu
Copy link
Contributor

heng-liu commented Oct 1, 2020

Looks like need to suppress CS0618 in some other files?

@heng-liu heng-liu merged commit b987ca9 into dev Oct 1, 2020
@heng-liu heng-liu deleted the dev-rrelyea-obsoletenewpackagesourceapis branch October 1, 2020 04:20
rrelyea pushed a commit that referenced this pull request Oct 1, 2020
* obsolete PackageSource related types from earlier in 5.8

* add pragmas to allow calling obsolete apis for now.

* additional pragmas

(cherry picked from commit b987ca9)
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