Skip to content
This repository was archived by the owner on Dec 14, 2018. It is now read-only.

Conversation

@blfuentes
Copy link

Related to Open issue:

#https://github.com/aspnet/Mvc/issues/6880

Removed obsolete override 'ActionResult.ExecuteResult' methods.

RedirectResult
LocalRedirectResult
RedirectToActionResult
RedirectToPageResult
RedirectToRouteResult

Related to Open issue: 
https://github.com/aspnet/Mvc/issues/6880

Removed obsolete override 'ActionResult.ExecuteResult' methods.

RedirectResult
LocalRedirectResult
RedirectToActionResult
RedirectToPageResult
RedirectToRouteResult
@dnfclas
Copy link

dnfclas commented Oct 24, 2018

CLA assistant check
All CLA requirements met.

@blfuentes
Copy link
Author

I get no errors while building the solution but it seems this is a breaking change for the tool.
========== Build: 69 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========

How should I proceed?

@pranavkm pranavkm changed the base branch from release/2.2 to master October 24, 2018 17:35
@pranavkm pranavkm closed this Oct 24, 2018
@pranavkm pranavkm reopened this Oct 24, 2018
@pranavkm
Copy link
Contributor

@blfuentes removing these methods may be a bit more involved. We'd most likely create a new baseline for the 3.0 milestone so you wouldn't see the error you're seeing. Thanks for your PR, but I'm going to close this since it's still early for us to figure out how to proceed with 3.0 breaking changes.

@pranavkm pranavkm closed this Oct 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants