Skip to content
This repository has been archived by the owner on Jun 16, 2024. It is now read-only.

Get or load plugin assembly (instead of always loading) #499

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ivvacula
Copy link

@ivvacula ivvacula commented May 6, 2020

When loading plugin type, check whether the plugin assembly is already loaded in the current AppDomain instead of calling AppDomain.Load right away.

This prevents AppDomain.Load from failing in cases when the plugin assembly is already loaded in the AppDomain and not present in the standard search locations.

…y loaded in the current AppDomain instead of calling AppDomain.Load right away.

This prevents AppDomain.Load from failing in cases when the plugin assembly is already loaded in the AppDomain and not present in the standard search locations.
@jordimontana82
Copy link
Owner

jordimontana82 commented May 6, 2020

Thanks a mill @ivvacula for this PR!

I'm flagging it as pending v2.x. Basically I'm in middle of porting FakeXrmEasy to .net core and all these PRs will reviewed as soon as v2.x is published.

@ivvacula
Copy link
Author

ivvacula commented May 6, 2020

Thanks a mill @ivvacula for this PR!

I'm flagging it as pending v2.x. Basically I'm in middle of porting FakeXrmEasy to .net and all these PRs will reviewed as soon as v2.x is published.

Thank you for the quick response, Jordi! Do you know when roughly will be the v2.x available? I don't need an exact date just a rule of thumb estimate (days, weeks, months?)

I plan on creating a few more PRs for small issues similar to this, so I'm wondering if it makes sense to wait with them for v2.x

@jordimontana82
Copy link
Owner

jordimontana82 commented May 7, 2020

Can't confirm yet, but looking like a matter of weeks hopefully, not days, and not months definitely.

Please keep them coming, the codebases will be slightly different but because 2.x will initially support client applications only (what the new CdsServiceClient nuget package supports), both 1.x and 2.x will have to coexist for some time... so still makes sense to keep adding the same features to both, until there is full parity.

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

Successfully merging this pull request may close these issues.

2 participants