Preparing report...

Report for github.com/heedy/pipescript

A+    Excellent!    Found 50 issues across 140 files

Tweet

gofmt97%

Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command


go_vet100%

go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.

No problems detected. Good job!


gocyclo95%

Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.

    • pipescript/object.go
    • Line 114: warning: cyclomatic complexity 29 of function (*oneToOneObjectTransform).Next() is high (> 15) (gocyclo)
    • Line 29: warning: cyclomatic complexity 18 of function (*aggregateObjectTransform).Next() is high (> 15) (gocyclo)

golint66%

Golint is a linter for Go source code.

    • pipescript/conversions.go
    • Line 59: warning: exported function ToString should have comment or be unexported (golint)
    • Line 83: warning: exported function Bool should have comment or be unexported (golint)
    • Line 114: warning: exported function FloatNoBool should have comment or be unexported (golint)
    • Line 129: warning: exported function IntNoBool should have comment or be unexported (golint)
    • Line 146: warning: exported function Equal should have comment or be unexported (golint)
    • pipescript/transforms/datetime/basics.go
    • Line 9: warning: exported var Hour should have comment or be unexported (golint)
    • Line 25: warning: exported var Day should have comment or be unexported (golint)
    • Line 41: warning: exported var Dayhour should have comment or be unexported (golint)
    • Line 57: warning: exported var Week should have comment or be unexported (golint)
    • Line 75: warning: exported var Weekday should have comment or be unexported (golint)
    • Line 91: warning: exported var Month should have comment or be unexported (golint)
    • Line 107: warning: exported var Monthday should have comment or be unexported (golint)
    • Line 120: warning: exported var Year should have comment or be unexported (golint)
    • Line 136: warning: exported var Yearmonth should have comment or be unexported (golint)
    • Line 152: warning: exported var Yearday should have comment or be unexported (golint)
    • pipescript/bytestreams/datapointarray.go
    • Line 11: warning: exported function NewArrayReader should have comment or be unexported (golint)
    • Line 21: warning: comment on exported type DatapointReader should be of the form "DatapointReader ..." (with optional leading article) (golint)
    • Line 34: warning: exported function NewDatapointReader should have comment or be unexported (golint)
    • pipescript/comparisons.go
    • Line 3: warning: exported var LtTransform should have comment or be unexported (golint)
    • Line 29: warning: exported var GtTransform should have comment or be unexported (golint)
    • Line 55: warning: exported var LteTransform should have comment or be unexported (golint)
    • Line 81: warning: exported var GteTransform should have comment or be unexported (golint)
    • Line 107: warning: exported var EqTransform should have comment or be unexported (golint)
    • Line 127: warning: exported var NeTransform should have comment or be unexported (golint)
    • pipescript/datapoint.go
    • Line 8: warning: exported type Datapoint should have comment or be unexported (golint)
    • Line 14: warning: exported method Datapoint.Float should have comment or be unexported (golint)
    • Line 22: warning: exported method Datapoint.Int should have comment or be unexported (golint)
    • Line 30: warning: exported method Datapoint.Bool should have comment or be unexported (golint)
    • Line 46: warning: exported method Datapoint.ToString should have comment or be unexported (golint)
    • Line 55: warning: exported method Datapoint.MapElement should have comment or be unexported (golint)
    • Line 64: warning: exported type DatapointArrayIterator should have comment or be unexported (golint)
    • Line 69: warning: exported method DatapointArrayIterator.Next should have comment or be unexported (golint)
    • Line 78: warning: exported function NewDatapointArrayIterator should have comment or be unexported (golint)
    • pipescript/pipe.go
    • Line 9: warning: exported type TransformEnv should have comment or be unexported (golint)
    • Line 14: warning: exported method TransformEnv.Next should have comment or be unexported (golint)
    • Line 30: warning: exported method TransformEnv.Peek should have comment or be unexported (golint)
    • Line 46: warning: exported type PipeElement should have comment or be unexported (golint)
    • Line 58: warning: exported function NewPipeElement should have comment or be unexported (golint)
    • Line 145: warning: exported method PipeElement.GetConst should have comment or be unexported (golint)
    • Line 172: warning: exported method PipeElement.Copy should have comment or be unexported (golint)
    • Line 203: warning: exported method PipeElement.Input should have comment or be unexported (golint)
    • Line 213: warning: exported method PipeElement.IsBasic should have comment or be unexported (golint)
    • Line 222: warning: exported method PipeElement.Next should have comment or be unexported (golint)
    • Line 226: warning: exported type Pipe should have comment or be unexported (golint)
    • Line 230: warning: exported function NewPipe should have comment or be unexported (golint)
    • Line 236: warning: exported function NewElementPipe should have comment or be unexported (golint)
    • Line 243: warning: exported function NewTransformPipe should have comment or be unexported (golint)
    • Line 260: warning: comment on exported method Pipe.InputIterator should be of the form "InputIterator ..." (golint)
    • Line 265: warning: exported method Pipe.Next should have comment or be unexported (golint)
    • Line 269: warning: exported method Pipe.Last should have comment or be unexported (golint)
    • Line 298: warning: exported method Pipe.Copy should have comment or be unexported (golint)
    • Line 306: warning: exported method Pipe.Join should have comment or be unexported (golint)
    • Line 314: warning: exported method Pipe.OneToOne should have comment or be unexported (golint)
    • Line 334: warning: exported method Pipe.IsBasic should have comment or be unexported (golint)
    • Line 350: warning: exported method Pipe.Simplify should have comment or be unexported (golint)
    • Line 447: warning: exported method Pipe.GetConst should have comment or be unexported (golint)
    • Line 458: warning: exported method Pipe.MarshalJSON should have comment or be unexported (golint)
    • Line 462: warning: exported method Pipe.UnmarshalJSON should have comment or be unexported (golint)
    • Line 494: warning: exported type ChanResult should have comment or be unexported (golint)
    • Line 499: warning: exported type ChannelPipe should have comment or be unexported (golint)
    • Line 504: warning: exported method ChannelPipe.Close should have comment or be unexported (golint)
    • Line 508: warning: exported function NewChannelPipe should have comment or be unexported (golint)
    • Line 543: warning: exported function MustPipe should have comment or be unexported (golint)
    • Line 551: warning: exported function MustParse should have comment or be unexported (golint)
    • pipescript/transform.go
    • Line 9: warning: exported type Iterator should have comment or be unexported (golint)
    • Line 17: warning: exported const ConstArgType should have comment (or a comment on this block) or be unexported (golint)
    • Line 37: warning: exported method ArgType.MarshalJSON should have comment or be unexported (golint)
    • Line 41: warning: exported method ArgType.UnmarshalJSON should have comment or be unexported (golint)
    • Line 63: warning: exported type TransformArg should have comment or be unexported (golint)
    • Line 71: warning: exported type Transform should have comment or be unexported (golint)
    • Line 82: warning: exported type TransformConstructor should have comment or be unexported (golint)
    • Line 84: warning: exported type TransformIterator should have comment or be unexported (golint)
    • pipescript/transform_constructors.go
    • Line 7: warning: exported type AggregatorFunc should have comment or be unexported (golint)
    • Line 8: warning: exported type Aggregator should have comment or be unexported (golint)
    • Line 16: warning: exported method Aggregator.Next should have comment or be unexported (golint)
    • Line 24: warning: exported method Aggregator.OneToOne should have comment or be unexported (golint)
    • Line 28: warning: exported function NewAggregator should have comment or be unexported (golint)
    • Line 39: warning: exported type BasicFunc should have comment or be unexported (golint)
    • Line 40: warning: exported type BasicInit should have comment or be unexported (golint)
    • Line 42: warning: exported type Basic should have comment or be unexported (golint)
    • Line 49: warning: exported method Basic.Next should have comment or be unexported (golint)
    • Line 60: warning: exported method Basic.OneToOne should have comment or be unexported (golint)
    • Line 64: warning: exported method Basic.GetConst should have comment or be unexported (golint)
    • Line 77: warning: exported function NewBasic should have comment or be unexported (golint)
    • Line 92: warning: exported type ArgBasicFunc should have comment or be unexported (golint)
    • Line 94: warning: exported type ArgBasic should have comment or be unexported (golint)
    • Line 101: warning: exported method ArgBasic.Next should have comment or be unexported (golint)
    • Line 112: warning: exported method ArgBasic.OneToOne should have comment or be unexported (golint)
    • Line 116: warning: exported method ArgBasic.GetConst should have comment or be unexported (golint)
    • Line 128: warning: exported function NewArgBasic should have comment or be unexported (golint)
    • Line 139: warning: exported type ConstIterator should have comment or be unexported (golint)
    • Line 143: warning: exported method ConstIterator.Next should have comment or be unexported (golint)
    • Line 154: warning: exported method ConstIterator.OneToOne should have comment or be unexported (golint)
    • Line 158: warning: exported function NewConstTransform should have comment or be unexported (golint)
    • pipescript/logical.go
    • Line 3: warning: exported var NotTransform should have comment or be unexported (golint)
    • Line 14: warning: exported var AndTransform should have comment or be unexported (golint)
    • Line 40: warning: exported var OrTransform should have comment or be unexported (golint)
    • pipescript/datasets/dataset.go
    • Line 17: warning: exported function NewDataset should have comment or be unexported (golint)
    • Line 26: warning: exported method Dataset.Reference should have comment or be unexported (golint)
    • Line 30: warning: exported method Dataset.Add should have comment or be unexported (golint)
    • Line 34: warning: exported method Dataset.Next should have comment or be unexported (golint)
    • Line 73: warning: exported function NewTDataset should have comment or be unexported (golint)
    • pipescript/datasets/schema.go
    • Line 12: warning: don't use underscores in Go names; type jsonSchema_prop should be jsonSchemaProp (golint)
    • Line 16: warning: exported type JSONSchema should have comment or be unexported (golint)
    • Line 25: warning: exported function NewSchema should have comment or be unexported (golint)
    • pipescript/buffer.go
    • Line 9: warning: exported var ErrBeforeStart should have comment or be unexported (golint)
    • Line 20: warning: exported type Buffer should have comment or be unexported (golint)
    • Line 33: warning: exported function NewBuffer should have comment or be unexported (golint)
    • Line 42: warning: exported method Buffer.Iterator should have comment or be unexported (golint)
    • Line 115: warning: exported type BufferIterator should have comment or be unexported (golint)
    • Line 131: warning: exported method BufferIterator.Next should have comment or be unexported (golint)
    • Line 164: warning: exported method BufferIterator.Peek should have comment or be unexported (golint)
    • Line 228: warning: exported method BufferIterator.Close should have comment or be unexported (golint)
    • Line 238: warning: exported type IteratorFromBI should have comment or be unexported (golint)
    • Line 242: warning: exported method IteratorFromBI.Next should have comment or be unexported (golint)
    • Line 253: warning: exported type EmptyIterator should have comment or be unexported (golint)
    • Line 255: warning: exported method EmptyIterator.Next should have comment or be unexported (golint)
    • pipescript/identity.go
    • Line 45: warning: exported var Identity should have comment or be unexported (golint)
    • Line 87: warning: exported var IdentityPipe should have comment or be unexported (golint)
    • Line 108: warning: exported var Dt should have comment or be unexported (golint)
    • Line 136: warning: exported var T should have comment or be unexported (golint)
    • pipescript/transforms/datetime/init.go
    • Line 11: warning: exported type TimeBasicFunc should have comment or be unexported (golint)
    • Line 37: warning: exported function NewTimeBasic should have comment or be unexported (golint)
    • Line 73: warning: exported function Register should have comment or be unexported (golint)
    • pipescript/arithmetic.go
    • Line 5: warning: exported var NegTransform should have comment or be unexported (golint)
    • Line 21: warning: exported var AddTransform should have comment or be unexported (golint)
    • Line 47: warning: exported var SubTransform should have comment or be unexported (golint)
    • Line 73: warning: exported var MulTransform should have comment or be unexported (golint)
    • Line 99: warning: exported var DivTransform should have comment or be unexported (golint)
    • Line 125: warning: exported var ModTransform should have comment or be unexported (golint)
    • Line 151: warning: exported var PowTransform should have comment or be unexported (golint)
    • pipescript/testing.go
    • Line 9: warning: exported type TestCase should have comment or be unexported (golint)
    • Line 17: warning: exported method TestCase.Run should have comment or be unexported (golint)

license100%

Checks whether your project has a LICENSE file.

No problems detected. Good job!


ineffassign99%

IneffAssign detects ineffectual assignments in Go code.


misspell99%

Misspell Finds commonly misspelled English words