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
Add preview package #7
Conversation
remove unused packages equivalent of PowerShell/PowerShell@d663b41
66f2868
to
84f69da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM just some nitpicks you don't have to fix
appveyor.yml
Outdated
Purpose: stable | ||
|
||
install: | ||
# - ps: Install-module pester -Scope CurrentUser -Force -SkipPublisherCheck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: any reason for this commented out code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
preview/snap/snapcraft.yaml
Outdated
version=$(curl -s 'https://raw.githubusercontent.com/PowerShell/PowerShell/master/tools/metadata.json' | jq .NextReleaseTag | sed 's/"//g' | sed 's/v//') | ||
echo $version | ||
fi | ||
summary: PowerShell for every system |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: GitHub has an exclamation mark at the end. Not sure if you want to add that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added
tools/releaseBuild/build.json
Outdated
"DockerImageName": "ps-snap-ubunutu-16-04", | ||
"BinaryBucket": "release" | ||
} | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: the indenting of this file is off.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
@@ -24,6 +32,8 @@ Push-Location | |||
try { | |||
Write-Verbose "snapcraft version $(snapcraft --version)" -Verbose | |||
Set-Location $location | |||
Set-location $directory |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: maybe just do Set-Location "$location\$directory"
or Set-Location "$location/$directory"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
LGTM! |
Any changes to the RFC will be addressed later |
Implements PowerShell/PowerShell-RFC#134
@tylerl0706 Can your review?