-
Notifications
You must be signed in to change notification settings - Fork 70
Lamar Migration #130
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
Lamar Migration #130
Conversation
nminaya
commented
Jan 21, 2025
- Replace StructureMap with Lamar due to its deprecation.
- Update projects .NET version
- .The minimum .net version supported for Lamar is .NET 6 - Install Lamar package
@nminaya Sorry for the delay in getting feedback here. As I went to merge this in, I noticed a few things I'd like us to fix:
I hope that makes sense, LMK if you have any questions or need help. |
@MattHoneycutt I went with the suggested approach but realized that now the code in SpecsFor.Lamar library is not covered with unit tests. Should I just create a SpecsFor.Lamar.Tests similar to SpecsFor.StructureMap.Tests? |
@nminaya Yeah, that's what we did for Autofac - there's a SpecsFor.Autofac.Tests project. That's probably something you could get Copilot Edits to generate, FYI. |