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

Delay first execution of ICommand.CanExecute(object parameter) until … #1179

Closed
wants to merge 1 commit into from

Conversation

MichaelVach
Copy link

…after initialization. Issue #316

Revert changes to Hyperlink.cs

…after initialization. Issue dotnet#316

Revert changes to Hyperlink.cs
@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Jul 5, 2019
@ghost ghost requested a review from SamBent July 5, 2019 08:47
@dnfclas
Copy link

dnfclas commented Jul 5, 2019

CLA assistant check
All CLA requirements met.

@vatsan-madhavan
Copy link
Member

@MichaelVach, Thanks for the contribution here - we appreciate it!

We are not accepting behavior changing PR's at this time with very rare exceptions (for e.g., something critical to the release of .NET Core 3.0) - so I'm closing this PR. Let's continue discussing #316 and agreeing on a fix approach that factors in app-compat concerns. Once you have a fix approach that is accepted by the product team and .NET 5 contributions have been opened up, we will be happy to reconsider PR's for this if it makes sense.

@ghost ghost locked as resolved and limited conversation to collaborators Apr 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants