Coverage remained the same at 79.676% when pulling ecff427 on Fedosin:fix_fmt into b971228 on gophercloud:master.

Build failed.

• gophercloud-unittest : SUCCESS in 4m 06s
• gophercloud-acceptance-test : FAILURE in 33m 29s
• gophercloud-acceptance-test-ironic : SUCCESS in 49m 29s

recheck

Build succeeded.

• gophercloud-unittest : SUCCESS in 4m 04s
• gophercloud-acceptance-test : SUCCESS in 2h 15m 43s
• gophercloud-acceptance-test-ironic : SUCCESS in 43m 25s

@Fedosin Thanks for doing this. This looks good to me. Let me know if you're ready to have this merged or if you were going to do any other work.

LGTM! 🐐

@jtopjian What about adding gofmt to the CI? And speaking of CI, do you have plans already for the after-Travis?

@pierreprinetti regarding gofmt, see my comments here: gophercloud/utils#133

This repo is currently using goimports to check for both code formatting and "import" statement stuff. The only area it's lacking is the -s flag for code simplification. For that, I'm on the fence. If someone wants to put the work into modifying the current script to do both goimports and gofmt, I'm happy to merge it. Otherwise, the basic formatting support that already exists is sufficient, IMO.

Regarding Travis... ugh. This is news to me. I'll have to look into it and see what options are available. Thank you for the heads-up.

@jtopjian yeah, the work (at least for this PR) is complete. we can merge it.