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

FOUNDATIONS: Assembly Service #6

Merged
merged 19 commits into from
May 30, 2024
Merged

Conversation

LBoullosa
Copy link
Member

@LBoullosa LBoullosa commented May 22, 2024

Closes: #5

@LBoullosa LBoullosa changed the title ShouldGetApplicationPathAssemblies -> FAIL FOUNDATIONS: Assembly Service May 22, 2024
@cjdutoit cjdutoit added the FOUNDATIONS The foundations category label May 22, 2024
Copy link
Member

@glhays glhays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LBoullosa @cjdutoit LGTM
As to what Christo mentioned in Discord do we need a couple of test cases for exceptions like DirectoryNotFound and FileNotFound.

@LBoullosa
Copy link
Member Author

@LBoullosa @cjdutoit LGTM As to what Christo mentioned in Discord do we need a couple of test cases for exceptions like DirectoryNotFound and FileNotFound.

@glhays FileNotFoundException is thrown by Assembly.Load, so yes, you are right we need to care about it.
DirectoryNotFound I don´t see it was thrown.

@LBoullosa LBoullosa marked this pull request as ready for review May 25, 2024 18:25
Copy link
Member

@glhays glhays left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a advantageous reason to convert the Assembly broker to asynchronous as to not block the constructor while the IO operations are being run?

@LBoullosa
Copy link
Member Author

Is there a advantageous reason to convert the Assembly broker to asynchronous as to not block the constructor while the IO operations are being run?
image

There is no available methods for loading an assembly asynchronously.
Even no available method for GetFiles asynchronously.

If this is a problem, then we should remove the logic inside the ctor, because no support for asynchronous ctor. Then the logic should be splitted and wrapped inside a Task.Run.

Let´s see if we can improve this, I am agree, but could be done in a second version.
What do you think?

@cjdutoit cjdutoit merged commit 9053f8c into main May 30, 2024
4 checks passed
@cjdutoit cjdutoit deleted the users/lboullosa/foundations-assembly branch May 30, 2024 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FOUNDATIONS The foundations category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FOUNDATIONS: AssemblyService
4 participants