Start rotating through all images - #17315
Conversation
James Truher (JamesWTruher)
left a comment
There was a problem hiding this comment.
one typo, one comment
| } elseif ($environment.IsFedora -or (Get-Command -Name dnf -CommandType Application -ErrorAction SilentlyContinue)) { | ||
| "dnf install -y -q" | ||
| } else { | ||
| } elseif (Get-Command -Name dnf -CommandType Application -ErrorAction SilentlyContinue) { |
There was a problem hiding this comment.
not sure about this - i think this line is essentially the same as line 1964. Are you looking for where there is no command dnf? - that would be !(get-command ...).
Co-authored-by: James Truher [MSFT] <jimtru@microsoft.com>
|
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
|
🎉 Handy links: |
* Use day of week for fallback seed * Use new properties to select CI image * move container selection code into * use parameter * fix azDO variable refernce * move container name left in non-PR build name * Use get-command to detect package manager * update dependencies in build.psm1 * Add RHEL to WSMan support function * Update build.psm1 * Update tools/ci.psm1 Co-authored-by: James Truher [MSFT] <jimtru@microsoft.com> Co-authored-by: James Truher [MSFT] <jimtru@microsoft.com>
PR Summary
PR Context
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:or[ WIP ]to the beginning of the title (theWIPbot will keep its status check atPendingwhile the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).