Skip to content
This repository was archived by the owner on Aug 17, 2021. It is now read-only.

Give PSArm block an alias-free environment so property keywords work #133

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

rjmholt
Copy link
Contributor

@rjmholt rjmholt commented Mar 31, 2021

Fixes #126.

In this PR, we use internal PowerShell engine APIs to tear down the aliases in each parent scope and store them while the Arm block body is executed. When execution is complete, we then restore the alias definitions to each scope using other internal PowerShell engine APIs.

daxian-dbw
daxian-dbw previously approved these changes Apr 12, 2021
Copy link
Member

@daxian-dbw daxian-dbw left a comment

Choose a reason for hiding this comment

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

The changes look fine, but can you please add more context information in the PR description about why an alias-free environment is needed? The referenced issue doesn't have the related context information either.

@rjmholt
Copy link
Contributor Author

rjmholt commented Apr 12, 2021

The changes look fine, but can you please add more context information in the PR description about why an alias-free environment is needed? The referenced issue doesn't have the related context information either.

I've updated the issue to describe the problem and the PR to describe the solution (the solution description not as detailed).

kilasuit
kilasuit previously approved these changes Apr 13, 2021
# Copyright (c) Microsoft Corporation.
# Licensed under the MIT License.


# Copyright (c) Microsoft Corporation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't need the duplicated Copyright line

# Copyright (c) Microsoft Corporation.
# Licensed under the MIT License.


# Copyright (c) Microsoft Corporation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't need the duplicated Copyright line

# Copyright (c) Microsoft Corporation.
# Licensed under the MIT License.


# Copyright (c) Microsoft Corporation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't need the duplicated Copyright line

@rjmholt rjmholt dismissed stale reviews from kilasuit and daxian-dbw via 26c8a5c April 13, 2021 16:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aliases should be removed for the duration of Arm blocks
3 participants