search

AutoMapper / AutoMapper.Extensions.Microsoft.DependencyInjection

MIT License
258 stars 79 forks source link

Remove automatic assembly scanning #90

Closed jbogard closed 5 years ago

jbogard commented 5 years ago

Seems like it just causes more problems than it's worth. We can keep the params one, but maybe add overloads that take one assembly, then params and assert.

lbargaoanu commented 5 years ago

Yes, the assemblies should be required. People can explicitly pass LoadedAssemblies if they want that.

lbargaoanu commented 5 years ago

Fixed by https://github.com/AutoMapper/AutoMapper.Extensions.Microsoft.DependencyInjection/commit/507b1ff19ed46104730b2c4eddfc28564cd259d3. I meant to open a PR, but I was not paying attention :) I guess we'll update the version when we upgrade the AM reference.

lock[bot] commented 5 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.