Skip to content

Merge pull request #985 from Tinyblargon/Fix#934 #16

Merge pull request #985 from Tinyblargon/Fix#934

Merge pull request #985 from Tinyblargon/Fix#934 #16

Triggered via push April 18, 2024 17:09
Status Success
Total duration 1m 25s
Artifacts

go.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 2 warnings
audit: proxmox/converters.go#L14
don't use underscores in Go names; func convert_KibibytesToString should be convertKibibytesToString
audit: proxmox/converters.go#L28
don't use underscores in Go names; func convert_SizeStringToKibibytes_Unsafe should be convertSizeStringToKibibytesUnsafe
audit: proxmox/converters.go#L46
don't use underscores in Go names; func parseSize_Unsafe should be parseSizeUnsafe
audit: proxmox/data_ha_group.go#L10
exported function DataHAGroup should have comment or be unexported
audit: proxmox/provider.go#L196
don't use underscores in Go names; var minimum_permissions should be minimumPermissions
audit: proxmox/provider.go#L268
if block ends with a return statement, so drop this else and outdent its block
audit: proxmox/provider.go#L274
don't use underscores in Go names; func parameter pm_api_url should be pmAPIURL
audit: proxmox/provider.go#L275
don't use underscores in Go names; func parameter pm_user should be pmUser
audit: proxmox/provider.go#L276
don't use underscores in Go names; func parameter pm_password should be pmPassword
audit: proxmox/provider.go#L277
don't use underscores in Go names; func parameter pm_api_token_id should be pmAPITokenID
audit
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-go@v2. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
audit
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-go@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/