Skip to content

Fix PSModuleAnalysisCachePath path for Windows images#515

Merged
TravisEz13 merged 4 commits into
PowerShell:masterfrom
kiazhi:pr-410
Nov 11, 2020
Merged

Fix PSModuleAnalysisCachePath path for Windows images#515
TravisEz13 merged 4 commits into
PowerShell:masterfrom
kiazhi:pr-410

Conversation

@kiazhi
Copy link
Copy Markdown
Contributor

@kiazhi kiazhi commented Oct 30, 2020

PR Summary

fix #410

  • Remove LOCALAPPDATA="C:\Users\ContainerAdministrator\AppData\Local"
  • Update PSModuleAnalysisCachePath="$LOCALAPPDATA\Microsoft\Windows\PowerShell\docker\ModuleAnalysisCache" to PSModuleAnalysisCachePath="C:\Users\Public\AppData\Local\Microsoft\Windows\PowerShell\docker\ModuleAnalysisCache" instead

PR Checklist

@kiazhi
Copy link
Copy Markdown
Contributor Author

kiazhi commented Oct 30, 2020

@PoshChan rerun CI

@PoshChan
Copy link
Copy Markdown
Collaborator

@kiazhi, I do not understand: run CI

Commands available in this repo for you:
  • retry <target> this will attempt to retry only the failed jobs for the target pipeline, restart can be used in place of retry
  • rebuild <target> this will perform a complete rebuild of the target pipeline, rerun can be used in place of rebuild Supported values for <target> which can be a comma separated list are: CI
  • get failures this will attempt to get the latest failures for all of the target pipelines
  • remind me in <value> <units> this will create a reminder that will be posted after the specified duration <value> is a number, and <units> can be minutes, hours, or days (singular or plural)

@kiazhi
Copy link
Copy Markdown
Contributor Author

kiazhi commented Oct 30, 2020

@PoshChan rerun CI

@PoshChan
Copy link
Copy Markdown
Collaborator

@kiazhi, successfully started rebuild of docker-PR

@kiazhi
Copy link
Copy Markdown
Contributor Author

kiazhi commented Oct 30, 2020

Whoopsies, something broke the pipe.

Looks like Pester module can't be found or install in one of the step in the pipeline.

@TravisEz13
Copy link
Copy Markdown
Member

The Gallery is having issue. I'll look to see if we can rerun tomorrow.

@TravisEz13
Copy link
Copy Markdown
Member

@PoshChan Please remind me in 1 day

@PoshChan
Copy link
Copy Markdown
Collaborator

@TravisEz13, this is the reminder you requested 1 day ago

@kiazhi
Copy link
Copy Markdown
Contributor Author

kiazhi commented Nov 3, 2020

@PoshChan rerun CI

@PoshChan
Copy link
Copy Markdown
Collaborator

PoshChan commented Nov 3, 2020

@kiazhi, successfully started rebuild of docker-PR

Copy link
Copy Markdown
Collaborator

@RDIL RDIL left a comment

Choose a reason for hiding this comment

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

Looks good, and CI passes now!

@TravisEz13 TravisEz13 merged commit 48b6ddf into PowerShell:master Nov 11, 2020
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.

Invalid environment variable LOCALAPPDATA

4 participants