Skip to content
This repository has been archived by the owner on Aug 1, 2023. It is now read-only.

openstack/compute/v2/extensions/adminactions: fix errors found by vet #640

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

functionary
Copy link

This pull request was made with an automated tool.

The suggested change fixes one or more problems discovered by "go vet".

See https://github.com/functionary/functionary for more details on
the @functionary GitHub user.

Vet found the following errors:
	openstack/compute/v2/extensions/adminactions/requests_test.go:38: result of fmt.Errorf call not used
	openstack/compute/v2/extensions/adminactions/requests_test.go:54: result of fmt.Errorf call not used
	openstack/compute/v2/extensions/adminactions/requests_test.go:70: result of fmt.Errorf call not used

These calls to fmt.Errorf should instead be calls to t.Errorf.
@coveralls
Copy link

coveralls commented Apr 19, 2018

Coverage Status

Coverage remained the same at 80.494% when pulling e19ac55 on functionary:autofix-7692 into e00690e on rackspace:master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants