m-lab / go

General purpose libraries / APIs for use in mlab code.
Apache License 2.0
5 stars 6 forks source link

Move LintMetrics to promtest package #43

Closed stephen-soltesz closed 5 years ago

stephen-soltesz commented 5 years ago

By importing the system "testing" package, any users of the prometheusx package inherited a long list of "testing" flags that would almost never be needed (see below) by users of prometheusx.MustStartPrometheus.

This change adds a new sub package of prometheusx named promtest that contains the LintMetrics function. No new code was added.

This change is not backward compatible.

2019/04/01 02:34:56 argsfromenv.go:38: Argument test.bench=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.benchmem=false
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.benchtime=1s
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.blockprofile=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.blockprofilerate=1
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.count=1
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.coverprofile=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.cpu=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.cpuprofile=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.failfast=false
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.list=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.memprofile=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.memprofilerate=0
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.mutexprofile=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.mutexprofilefraction=1
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.outputdir=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.parallel=4
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.run=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.short=false
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.testlogfile=/var/log/...
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.timeout=0s
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.trace=
2019/04/01 02:34:56 argsfromenv.go:38: Argument test.v=true

This change is Reviewable

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 376


Totals Coverage Status
Change from base Build 374: 0.0%
Covered Lines: 406
Relevant Lines: 564

💛 - Coveralls