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

File_Adapter: added missing copy local and build event for System.Management.Automation.dll #197

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

alelom
Copy link
Member

@alelom alelom commented Sep 11, 2024

Issues addressed by this PR

Closes #196

Added missing copy local and build event for System.Management.Automation.dll, from its original location sourced from C:\WINDOWS\Microsoft.Net\assembly\GAC_MSIL\System.Management.Automation\v4.0_3.0.0.0__31bf3856ad364e35\System.Management.Automation.dll.

Test files

Follow the File_Toolkit testing procedure

Changelog

Additional comments

@alelom alelom added the type:bug Error or unexpected behaviour label Sep 11, 2024
@alelom alelom requested a review from Dmamedova September 11, 2024 12:14
@alelom alelom self-assigned this Sep 11, 2024
@alelom alelom requested a review from michaelhoehn September 11, 2024 14:35
@alelom alelom changed the title File_Adapter: added missing copy local and build event for System.Management.Automation File_Adapter: added missing copy local and build event for System.Management.Automation.dll Sep 11, 2024
Copy link

@michaelhoehn michaelhoehn left a comment

Choose a reason for hiding this comment

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

I've compiled the code and can confirm the file is copied to the BHoM Assemblies folder as expected. I have not run any further tests.

Copy link

@Dmamedova Dmamedova left a comment

Choose a reason for hiding this comment

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

Because I don't have Visual Studio installed yet, with @alelom we manually copied the missing assembly in the BHoM installation folder and that fixed the issue for me. The File_Adapter is now correctly doing the Pull, while before it was giving me an error of missing assembly. We have run test scripts and they work correctly.

@alelom
Copy link
Member Author

alelom commented Sep 11, 2024

The combined reviews of @michaelhoehn and @Dmamedova make this PR safe to merge.

@alelom alelom merged commit 7f58785 into develop Sep 11, 2024
@alelom alelom deleted the File_Adapter-#196-missingCopyDll branch September 11, 2024 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Error or unexpected behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

File_Adapter: missing copy local and build event for System.Management.Automation
3 participants