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

Remove deprecated API #2798

Merged
merged 1 commit into from
Feb 7, 2020
Merged

Remove deprecated API #2798

merged 1 commit into from
Feb 7, 2020

Conversation

RussKie
Copy link
Member

@RussKie RussKie commented Feb 3, 2020

As identified in #2476 (comment)

Relates to #2091

Proposed changes

  • Remove the following API because they have been obsolete since .NET 2.0:
    • StatusBar
    • StatusBarDrawItemEventArgs
    • StatusBarDrawItemEventHandler
    • StatusBarPanel
    • StatusBarPanelAutoSize
    • StatusBarPanelBorderStyle
    • StatusBarPanelClickEventArgs
    • StatusBarPanelClickEventHandler
    • StatusBarPanelStyle

Customer Impact

  • Removed controls no longer available to customers and will cause builds to break
  • Customers will need to replace the removed controls with available alternatives.

Regression?

  • Yes / No

Risk

  • Medium, the controls in questions have been deprecated in .NET 2.0 era, however there is a recorded use of those.

Test methodology

  • TBD
Microsoft Reviewers: Open in CodeFlow

@RussKie RussKie added 📖 documentation: breaking A change in behavior that could be breaking for applications. Needs to be documented. Servicing-consider .NET Shiproom label indicating a PR seeks to enter into a branch under Tell-Mode criteria labels Feb 3, 2020
@RussKie RussKie requested a review from a team as a code owner February 3, 2020 06:02
@RussKie RussKie self-assigned this Feb 3, 2020
@RussKie RussKie added api-suggestion (1) Early API idea and discussion, it is NOT ready for implementation and removed Servicing-consider .NET Shiproom label indicating a PR seeks to enter into a branch under Tell-Mode criteria labels Feb 3, 2020
@codecov
Copy link

codecov bot commented Feb 3, 2020

Codecov Report

Merging #2798 into master will decrease coverage by 0.05216%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##             master       #2798         +/-   ##
==================================================
- Coverage   59.4119%   59.35974%   -0.05216%     
==================================================
  Files          1243        1236          -7     
  Lines        431284      428762       -2522     
  Branches      38995       38757        -238     
==================================================
- Hits         256234      254512       -1722     
+ Misses       169639      168902        -737     
+ Partials       5411        5348         -63
Flag Coverage Δ
#Debug 59.35974% <ø> (-0.05217%) ⬇️
#production 32.12776% <ø> (-0.06927%) ⬇️
#test 98.97632% <ø> (+0.0049%) ⬆️

JuditRose
JuditRose previously approved these changes Feb 3, 2020
@merriemcgaw
Copy link
Member

merriemcgaw commented Feb 6, 2020

Business case approved at my level. LGTM

merriemcgaw
merriemcgaw previously approved these changes Feb 6, 2020
- Remove the following API because they have been obsolete since .NET 2.0:
	* `StatusBar`
	* `StatusBarDrawItemEventArgs`
	* `StatusBarDrawItemEventHandler`
	* `StatusBarPanel`
	* `StatusBarPanelAutoSize`
	* `StatusBarPanelBorderStyle`
	* `StatusBarPanelClickEventArgs`
	* `StatusBarPanelClickEventHandler`
	* `StatusBarPanelStyle`

Relates to #2091
@RussKie RussKie dismissed stale reviews from merriemcgaw and JuditRose via 827db4f February 6, 2020 23:34
@RussKie RussKie force-pushed the dev/igveliko/Deprecate_StatusBar branch from a80955d to 827db4f Compare February 6, 2020 23:34
@RussKie RussKie merged commit 804091a into master Feb 7, 2020
@RussKie RussKie deleted the dev/igveliko/Deprecate_StatusBar branch February 7, 2020 01:08
@ghost ghost added this to the 5.0 milestone Feb 7, 2020
@RussKie
Copy link
Member Author

RussKie commented Feb 7, 2020

@OliaG FYI

@RussKie
Copy link
Member Author

RussKie commented Feb 7, 2020

@vatsan-madhavan FYI, I hope it doesn't affect WPF

@vatsan-madhavan
Copy link
Member

vatsan-madhavan commented Feb 7, 2020

FYI, I hope it doesn't affect WPF

You can just test for it instead and avoid breaking things after merging them, and then have to worry about reverting them or needing the next-repo to do stuff in a hurry etc.

  • build you repo locally to get Microsoft.Private.Winforms.5.0.0-dev.nupkg under, say %winformsroot%\artifacts\packages\Debug\NonShipping
  • clear nuget cache (nuget locals all -clear)
  • clone dotnet/wpf (git clone https://github.com/dotnet/wpf)
    • edit NuGet.config and add a new source pointing to %winformsroot%\artifacts\packages\Debug\NonShipping
    • darc update-dependency -n Microsoft.Private.Winforms -v 5.0.0-dev
    • build wpf
    • build -pack -configuration Debug|Release -platform x86|x64

Or better yet, you can set up a couple of test branches on the internal Azure pipelines mirror repos to and use the build-pipelines to run the builds. For complex changes that might break the next repo, we do this sort of stuff proactively all the time. See this for example.

/cc @rladuca

@RussKie
Copy link
Member Author

RussKie commented Feb 7, 2020

Yes, an after thought on my part.
Thank you for the instructions

@RussKie
Copy link
Member Author

RussKie commented Feb 7, 2020

Docs: dotnet/docs#17008

@ghost ghost locked as resolved and limited conversation to collaborators Feb 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api-suggestion (1) Early API idea and discussion, it is NOT ready for implementation 📖 documentation: breaking A change in behavior that could be breaking for applications. Needs to be documented.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants