Skip to content

Commit 5cfb5a3

Browse files
committed
lint + comment
1 parent 6053bf5 commit 5cfb5a3

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,7 @@ upload-installer:
102102
.PHONY: lint-ps
103103
lint-ps:
104104
# I installed from .deb in releases here https://github.com/PowerShell/PowerShell
105+
# note: this linter is not incredibly useful; for example it won't catch some missing reference errors. Don't over-rely
105106
pwsh -Command 'if (-not (Get-Module PSScriptAnalyzer -ListAvailable)) { Install-Module PSScriptAnalyzer -Scope CurrentUser -Force -ErrorAction Stop }; Invoke-ScriptAnalyzer -Path windows-installer-agent.ps1 -Settings PSScriptAnalyzerSettings.psd1 -Severity Warning,Error -EnableExit; Invoke-ScriptAnalyzer -Path windows-uninstall-agent.ps1 -Settings PSScriptAnalyzerSettings.psd1 -Severity Warning,Error -EnableExit'
106107

107108
.PHONY: windows-installer

version_control.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,6 @@ func (c *VersionCache) UpdateBinary(ctx context.Context, binary string) (bool, e
347347
expectedMimes = []string{"application/vnd.microsoft.portable-executable"}
348348
}
349349

350-
//nolint:goconst
351350
if runtime.GOOS == "darwin" {
352351
expectedMimes = []string{"application/x-mach-binary"}
353352
}
@@ -409,7 +408,6 @@ func (c *VersionCache) getProtectedFilesAndCleanVersions(ctx context.Context, ma
409408
// add protection for the current symlinked binaries
410409
for _, path := range []string{"viam-agent", "viam-server"} {
411410
if runtime.GOOS == "windows" {
412-
//nolint:goconst
413411
path += ".exe"
414412
}
415413

0 commit comments

Comments
 (0)