-
Notifications
You must be signed in to change notification settings - Fork 57
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Cleans up a number of small non-functional issues scattered around the repo. - update all dependencies to the latest - adds a functional test with most options enabled at once - clean up warnings in time parsing, rework error handling, eliminate old errors package - delete some dead code & docs - remove names from unused func parameters - finish removing the old protobuf repo Detailed commits follow: * add a fully-loaded CLI test Looking into #316 I realized there wasn't a test with most of the command-line options in play, including a --attrs on a simple span. This adds one that's mostly complete. Could check a few more things but this is a start. * update dependencies validated with `go build && go test`, looks good * (lint) remove named param to clear warning The config is unused in generating completion. Removed its name and now the linters are happy again. * rework errors in config Started by cleaning up a warning from the nil checking linter and ended up removing the old errors package and rewrapping errors to be both more correct and hopefully a bit easier to follow. * remove names from unused func params to clear warnings * remove dead code * remove errors package Replaced with calls to fmt.Errorf() * cleanup docs + update CHANGELOG.md * remove last reference to github.com/golang/protobuf cleans up go.mod a bit more
- Loading branch information
Showing
10 changed files
with
153 additions
and
109 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.