-
Notifications
You must be signed in to change notification settings - Fork 255
Add messages to obsolete specs for clarity #373
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 messages to obsolete specs for clarity #373
Conversation
Hi @Rowe2ryWA, Thank you for taking the time to contribute here.
|
Yes, I will make the edits and update the branch. |
Thanks! |
Super simple contribution but hopefully a helpful one.
I noticed the obsolete warnings when working in our organization's server repo and it took 15 minutes of research and git blame to find the commit where these were marked obsolete, study the code and determine what I should have been doing instead.
I figured I'd contribute to the project to save the next guy some time. Ardalis is a major backbone of our project so I hope this contribution helps. 🙂