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

Add extra information to obsolete messages #681

Closed
dotnetjunkie opened this issue Apr 5, 2019 · 1 comment
Closed

Add extra information to obsolete messages #681

dotnetjunkie opened this issue Apr 5, 2019 · 1 comment

Comments

@dotnetjunkie
Copy link
Collaborator

dotnetjunkie commented Apr 5, 2019

Every version of Simple Injector contains obsolete messages, but the messages don't always describe when the Obsolete attribute will be treated as error and when the feature will be removed completely from the library.

It is, therefore, useful to add the following information to obsolete messages:

Will be treated as an error from version {version}. Will be removed in version {version}

Perhaps even add a unit test that verifies whether future messages comply with this rule.

Also, messages should not use any of the following redundant information (inspired by a Fody plugin):

  • Do not call this method
  • This method will be removed
  • This method is obsolete
  • The replacement method is
@dotnetjunkie dotnetjunkie added this to the Backlog milestone Apr 5, 2019
@dotnetjunkie dotnetjunkie modified the milestones: Backlog, v4.6 Apr 22, 2019
@dotnetjunkie
Copy link
Collaborator Author

Feature-681 branch created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant