Skip to content

Request: Terminal Change "Run Code" #4424

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
2 tasks done
Shxpard opened this issue Feb 17, 2023 · 3 comments
Closed
2 tasks done

Request: Terminal Change "Run Code" #4424

Shxpard opened this issue Feb 17, 2023 · 3 comments
Labels
Issue-Enhancement A feature request (enhancement). Resolution-Answered Will close automatically.

Comments

@Shxpard
Copy link

Shxpard commented Feb 17, 2023

Prerequisites

  • I have written a descriptive issue title.
  • I have searched all issues to ensure it has not already been reported.

Summary

I would like to have the terminal show:
PS> C:\Example
Hello World

Instead of:
PS>
Run
PS>. 'C:\Example'
Hello World
PS> C:\
Run
PS>. 'C:\Example'
Hello World
PS> C:\

This occurs with the ISE and works with v2021.12.0.
However with the newer version of the Powershell Extension it seems to really clutter the output.
WantedBehavior
UnwantedBehavior

Proposed Design

I have no idea how the design of this works. I do think there was some intention behind the change but figured I would post this. I'm not a power user as others may be, so I'm not sure if this aesthetic design is actually practical or not.

@Shxpard Shxpard added the Issue-Enhancement A feature request (enhancement). label Feb 17, 2023
@ghost ghost added the Needs: Triage Maintainer attention needed! label Feb 17, 2023
@andyleejordan
Copy link
Member

Hi, the change is to show exactly what's bring run. In this case, a dot-source of the file (as opposed to an ampersand-call, like #4327 asks to make as an option). As it is, the implementation is simply just showing what exactly is being run, so we don't really want to add any special logic to hide that . prefix if (and only if!) it was requested by the Run Code command. It's less aesthetics and more...precision. Does that make sense?

@andyleejordan andyleejordan removed the Needs: Triage Maintainer attention needed! label Feb 21, 2023
@Shxpard
Copy link
Author

Shxpard commented Feb 23, 2023

Of course. I expected as much that it would be for precise for users. Thank you for your help.

@andyleejordan andyleejordan added the Resolution-Answered Will close automatically. label Feb 23, 2023
@ghost ghost closed this as completed Feb 24, 2023
@ghost
Copy link

ghost commented Feb 24, 2023

This issue has been marked as answered and has not had any activity in a day. It has been automatically closed for housekeeping purposes.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Enhancement A feature request (enhancement). Resolution-Answered Will close automatically.
Projects
None yet
Development

No branches or pull requests

2 participants