Skip to content

Add version to PowerShell banner - #3916

Merged
Jason Shirk (lzybkr) merged 2 commits into
PowerShell:masterfrom
iSazonov:banner-version
Jun 7, 2017
Merged

Add version to PowerShell banner#3916
Jason Shirk (lzybkr) merged 2 commits into
PowerShell:masterfrom
iSazonov:banner-version

Conversation

@iSazonov

@iSazonov Ilya (iSazonov) commented Jun 2, 2017

Copy link
Copy Markdown
Collaborator

Close #3895

Motivation

Since we support side-by-side, it makes it easier to see which PowerShell version you started rather than checking $psversiontable or $pshome.

Fix

Add "GitCommitId" to PowerShell banner.

Follow-up work

Based on #3690 later we will change a GitCommitId format.

<value>PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.</value>
Copyright (C) Microsoft Corporation. All rights reserved.
Version: {0}</value>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think the version proposed by Steve Lee (@SteveL-MSFT) originally reads nicer:

PowerShell {0}
Copyright (C) Microsoft Corporation. All rights reserved.

For example, here is how python banner looks like

Python 2.7.10 (default, Oct 18 2015, 00:31:39) 
[GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-700.0.72)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think it doesn't look elegant :-) but fixed.

@SteveL-MSFT Steve Lee (SteveL-MSFT) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@iSazonov

Copy link
Copy Markdown
Collaborator Author

sergei (@vors) Can you approve and merge?

@lzybkr
Jason Shirk (lzybkr) merged commit 0052ebb into PowerShell:master Jun 7, 2017
@iSazonov
Ilya (iSazonov) deleted the banner-version branch June 8, 2017 05:11
@iSazonov
Ilya (iSazonov) restored the banner-version branch June 20, 2017 17:37
@iSazonov
Ilya (iSazonov) deleted the banner-version branch July 20, 2017 11:35
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
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.

5 participants