8000 Use Chocolatey scripting best practices for Boxstarter Chocolatey packages · Issue #507 · chocolatey/boxstarter · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Use Chocolatey scripting best practices for Boxstarter Chocolatey packages #507
Open
@gep13

Description

@gep13

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

No one assigned

    Labels

    0 - BacklogIssue is accepted, but is not ready to be worked on or not in current sprintImprovementIssues that enhances existing functionality, or adds new features

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0