[pull] master from PowerShell:master - #368
Open
pull[bot] wants to merge 2795 commits into
Open
Conversation
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: TravisEz13 <10873629+TravisEz13@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: anamnavi <annavied@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ous pattern (#26429) Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ipeline (#27667) 1. Create the `LocProject.json` file for PowerShell localization. 2. Remove the unused `.resx` resource files and source files. 3. Update the `ResourceValidation` tests.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Justin Chung <chungjustin@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4d8b066a-3f02-4ae4-bfd5-c08b6a180beb
`Test-Bom` was only used in `New-MSIPackage`, which was removed in #27213 as we are not building MSI packages for v7.7 and onward. `Test-Bom` is no longer used anywhere in PowerShell repo, so removing it and its helpers, as well as the boms JSON files, in this PR.
This PR essentially reverts #3331, which updated all W3C URIs in the `.resx` files from `http` to `https`. That change turned out to be wrong. Those W3C URIs are namespace identifiers, not fetchable URLs -- they must be byte‑exact. Changing them to use `https` corrupted them, and thus `ManagedParser` in the localization pipeline can't recognize the schema, which resulted in the `BL1024 "child table to itself"` error. - This PR makes pure, mechanical 2-line substitution in every `.resx` file: `https://www.w3.org → http://www.w3.org`. - The `ClipboardResources.resx` was using Schema version 1.3, and it's changed to version 2.0 as the rest `.resx` files in this PR.
…0: Build ID 2652124 (#27736)
…ed in satellite assemblies (#27756)
…0: Build ID 2660148 (#27752)
…0: Build ID 2661935 (#27766)
…se pipelines (#27795) Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Justin Chung <124807742+jshigetomi@users.noreply.github.com>
…0: Build ID 2670960 (#27802)
…0: Build ID 2674492 (#27821)
Co-authored-by: Justin Chung <chungjustin@microsoft.com> Copilot-Session: 8784f802-9d66-4692-b64d-c0713fa213cb
Co-authored-by: Justin Chung <chungjustin@microsoft.com> Copilot-Session: 8784f802-9d66-4692-b64d-c0713fa213cb
Co-authored-by: Justin Chung <124807742+jshigetomi@users.noreply.github.com>
The method `GetLookupDirectoryPaths` always returns the cached instance of path collection. However, the returned collection gets mutated in `CommandPathSearch.ResolveCurrentDirectoryInLookupPaths` to resolve the relative paths such as `.\tools` based on the user's current working directory `$PWD`, so for example, `.\tools` gets replaced with `cwd-1\tools` in the cached instance. Then, when the user changes to a different working directory `cwd-2`, command discovery won't find the executable or ps1 script under `cwd-2/tools` as expected, but those executables under `cwd-1\tools` will always be discoverable no matter what the `$PWD` is. That behavior is incorrect. This pull request improves the caching logic of `GetLookupDirectoryPaths` and makes it return a copy of the path collection, so the mutation happens to the returned copy, and the cached instance is kept intact. So, for every command search, the `CommandPathSearch` will resolve relative paths against the `$PWD` that the user is located at that time.
…s and also enable tests (#27773) - Fix the scripts used for `Copy-Item -FromSessoin/-ToSession` and `PSEdit`. They use `-Encoding Byte`, which only works in PS5.1. In PS 7+, we need to use `-AsByteStream` instead. So, those scenarios (copy alternate data stream to/from remote PS 7 session and edit file from a remote PS 7 session) have been broken for a long time. - The remote tests for `Copy-Item` have always been disabled. This PR re-enables those tests for Windows (the `HelperRemoting` module only works on Windows).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )