devcontainers: mount workspace in /PowerShell by rzippo · Pull Request #23857 · PowerShell/PowerShell · GitHub
Skip to content

devcontainers: mount workspace in /PowerShell#23857

Merged
TravisEz13 merged 3 commits intoPowerShell:masterfrom
rzippo:fix-devcontainers-packaging
Jul 31, 2024
Merged

devcontainers: mount workspace in /PowerShell#23857
TravisEz13 merged 3 commits intoPowerShell:masterfrom
rzippo:fix-devcontainers-packaging

Conversation

@rzippo
Copy link
Copy Markdown
Contributor

@rzippo rzippo commented May 27, 2024

PR Summary

This PR fixes #23854.
To build packages, Start-PSPackage expects the repository to be at /Powershell, but devcontainers mount it by default at /workspaces/name-of-the-folder.
This fix configures the devcontainers to mount the workspace at /Powershell, so that Start-PSPackage works as expected.

The fix was tested by building a 7.4.2 package on the Ubuntu 18.04 devcontainer (see this branch).

PR Context

PR Checklist

@rzippo
Copy link
Copy Markdown
Contributor Author

rzippo commented May 27, 2024

Copy link
Copy Markdown
Collaborator

@kilasuit kilasuit left a comment

Choose a reason for hiding this comment

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

LGTM

fedora is not supported by PowerShell anymore
Update to a supported version of Ubuntu
@TravisEz13 TravisEz13 merged commit 2b64ee6 into PowerShell:master Jul 31, 2024
@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Review - Needed The PR is being reviewed label Jul 31, 2024
@TravisEz13 TravisEz13 added Review - Needed The PR is being reviewed CL-Tools Indicates that a PR should be marked as a tools change in the Change Log labels Jul 31, 2024
chrisdent-de pushed a commit to chrisdent-de/PowerShell that referenced this pull request Sep 12, 2024
* devcontainers: mount workspace in /PowerShell

* Delete .devcontainer/fedora30 directory

fedora is not supported by PowerShell anymore

* Update Dockerfile

Update to a supported version of Ubuntu

---------

Co-authored-by: Travis Plunk <travis.plunk@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log Review - Needed The PR is being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot build packages from devcontainer

3 participants