Skip to content

Conversation

@axtens
Copy link
Member

@axtens axtens commented Mar 29, 2023

Installs Pester if not found. Updates otherwise.

@axtens
Copy link
Member Author

axtens commented Mar 29, 2023

 Line |
   2 |      Update-Module -Name Pester
     |      ~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Module 'Pester' was not installed by using Install-Module, so it cannot be updated.

So that would imply that the Windows instance already has Pester installed. Would it be better then to have Pester installed in the Linux version the way it is in the Windows?

@ErikSchierboom
Copy link
Member

I have no idea basically, as I don't really know much about PowerShell

@axtens
Copy link
Member Author

axtens commented Mar 29, 2023

In the short term, we can install if needed and it will never be needed in the Windows version because (I assume) the Dockerfile has specified to install it for Windows. In the long term, I'll see whether the Dockerfile can be told what to do for Linux.

@axtens
Copy link
Member Author

axtens commented Mar 29, 2023

Okay, passing all checks.

@ErikSchierboom ErikSchierboom merged commit 107601d into exercism:main Mar 29, 2023
@axtens axtens deleted the pester branch March 30, 2023 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants