-
Notifications
You must be signed in to change notification settings - Fork 13
Install PSScriptAnalyzer only if it is not there yet #5
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
Conversation
|
@bb-froggy Want to extend this to ConvertoToSARIF also while at it? 🙂 |
|
Good idea, I have added the code for ConvertToSARIF and tested that it works in my environments. |
|
Thanks. Trying to reduce CI time by caching these, so the always force install is not great. Let's hope this gets merged at some point 🙂 |
|
Hi, when do you think this will be merged? Currently I am using the commit from @bb-froggy for ubuntu-latest: Run microsoft/psscriptanalyzer-action@452b6222221f82be3420596e9bd148a7b2bb5bb8
Run $analyzerModule = Get-Module -ListAvailable -Name PSScriptAnalyzer
Modules installed, now running tests. |
|
Any chance this is going to be merged? Consistently getting the following error in my workflow Run microsoft/psscriptanalyzer-action@2044ae068e37d0161fa2127de04c19633882f061
Run Install-Module -Name PSScriptAnalyzer -Force
WARNING: The version '1.20.0' of module 'PSScriptAnalyzer' is currently in use. Retry the operation after closing the applications. |
|
@JamesWTruher Got a tip that you are a maintainer of this repo. Is anyone watching this and ConvertToSARIF? Would it be possible to get the pending PRs merged? Btw. this repo also contains some template-content in README (project-section) and SUPPORT. |
|
@fflaten I am not the maintainer of this repo |
Oh, bad tip then. Are you by any chance able to see who is or know who might be able to assist? |
|
Maybe @A-Katopodis ? Are you a maintainer? |
|
Are you able to get someone's attention to these repos @SydneyhSmith ? 😇 Also, should they be part of Powershell-org like PSScriptAnalyzer itself? |
|
Hi folks, Sorry for the delay. I did take a look and I will merge and create a PR to update the default Github actions as well. Thanks! |
|
Thanks @A-Katopodis. I appreciate it 🙂 |
This avoids an error message when PSScriptAnalyzer is already there (the default in current GitHub Actions using ubuntu-latest).
Also changed the test message to a more production-compatible one.