Skip to content

Conversation

owais
Copy link
Contributor

@owais owais commented Jul 3, 2019

Also ran go mod tidy

Fixes #102

songy23
songy23 previously approved these changes Jul 3, 2019
bogdandrutu
bogdandrutu previously approved these changes Jul 3, 2019
Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a corresponding exclusion of sources on Makefile, if that is not required anymore please remove it (it pointed to ./tools/* iirc)

// https://github.com/golang/go/wiki/Modules#how-can-i-track-tool-dependencies-for-a-module
// on how to add tooling dependencies.
package tools
// on how to ping add tooling dependencies to a go.mod file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"ping add"?

@codecov-io
Copy link

codecov-io commented Jul 3, 2019

Codecov Report

Merging #108 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #108   +/-   ##
=======================================
  Coverage   69.77%   69.77%           
=======================================
  Files         101      101           
  Lines        6392     6392           
=======================================
  Hits         4460     4460           
  Misses       1698     1698           
  Partials      234      234

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ce4809...d344f07. Read the comment docs.

Also ran `go mod tidy`

Fixes #102
@owais owais dismissed stale reviews from bogdandrutu and songy23 via d344f07 July 3, 2019 18:23
@flands flands added this to the 0.1.0 milestone Jul 3, 2019
@bogdandrutu bogdandrutu merged commit 7ff068a into open-telemetry:master Jul 3, 2019
@owais owais deleted the move-tools-pkg branch July 3, 2019 19:27
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
…metry#108)

* Add a YAML getting started to deploy splunk-otel-collector

Example is only for gateway mode.

Also update README and consolidate Docker docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Explain why do we need a tools package

7 participants