PLT-1.1 adding breakout config code · openconfig/featureprofiles@853efa9

Run revive: internal/otg_helpers/otg_config_helpers/otgconfighelpers.go#L96

parameter 't' seems to be unused, consider removing or renaming it as _

Run revive: feature/platform/tests/breakout_configuration/breakout_base_test.go#L1

don't use an underscore in package name

Run revive: feature/platform/tests/breakout_configuration/breakout_base_test.go#L224

if block ends with a return statement, so drop this else and outdent its block

Run revive: feature/platform/tests/breakout_configuration/breakout_test.go#L31

should omit type bool from declaration of var foundExpectedInterfaceFlag; it will be inferred from the right-hand side

Run revive: feature/platform/tests/breakout_configuration/breakout_test.go#L255

this block is empty, you can remove it

Run revive: feature/platform/tests/breakout_configuration/breakout_test.go#L1

don't use an underscore in package name

Run revive: feature/platform/tests/breakout_configuration/breakout_test.go#L98

var ateIpAddress should be ateIPAddress

Run revive: feature/platform/tests/breakout_configuration/breakout_test.go#L104

var gwIp should be gwIP

Run revive: feature/platform/tests/breakout_configuration/breakout_test.go#L105

var dutIpAddress should be dutIPAddress

Run revive: feature/platform/tests/breakout_configuration/breakout_test.go#L156

struct field dutIntfIp should be dutIntfIP

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

Provide feedback