-
Notifications
You must be signed in to change notification settings - Fork 164
Open
Labels
0 - BacklogIssue is accepted, but is not ready to be worked on or not in current sprint.Issue is accepted, but is not ready to be worked on or not in current sprint.EnhancementIssues that introduce new functionality to the project, or enhances/updates existing functionality.Issues that introduce new functionality to the project, or enhances/updates existing functionality.
Description
Is Your Feature Request Related To A Problem? Please describe.
There are instances where scripting best practices are not being following in the packaging scripts for the Boxstarter packages. For example, in the main Boxstarter package there is:
try {
Write-Host "To load all Boxstarter Modules immediately, just enter 'BoxstarterShell'." -ForegroundColor Yellow
Write-Host "Interested in Windows Azure VM integration? Run CINST Boxstarter.Azure to install Boxstarter's Azure integration."
} catch {
throw $_.exception
}
when this should be:
Write-Warning "To load all Boxstarter Modules immediately, just enter 'BoxstarterShell'."
Write-Host "Interested in Windows Azure VM integration? Run CINST Boxstarter.Azure to install Boxstarter's Azure integration."
Describe The Solution. Why is it needed?
Follow the recommended best practices for packaging scripts.
A full audit of all the packages should be completed.
Additional Context.
N/A
Related Issues
N/A
Metadata
Metadata
Assignees
Labels
0 - BacklogIssue is accepted, but is not ready to be worked on or not in current sprint.Issue is accepted, but is not ready to be worked on or not in current sprint.EnhancementIssues that introduce new functionality to the project, or enhances/updates existing functionality.Issues that introduce new functionality to the project, or enhances/updates existing functionality.