Skip to content
This repository was archived by the owner on Aug 17, 2020. It is now read-only.

Conversation

@tonyredondo
Copy link
Contributor

This PR improves support for projects without go modules file (ex: https://github.com/Netflix/chaosmonkey) by exporting SourceRoot from the agent metadata:

Before:
image

After:
image

@tonyredondo tonyredondo requested a review from fermayo February 10, 2020 13:26
@ghost ghost assigned tonyredondo Feb 10, 2020
@tonyredondo tonyredondo changed the title Support for project without go.mod Support for projects without go.mod Feb 10, 2020
@tonyredondo tonyredondo force-pushed the no-module-package-name branch from c7b2483 to 2725cba Compare February 12, 2020 13:06
@tonyredondo tonyredondo force-pushed the no-module-package-name branch 4 times, most recently from c4badc1 to 7c04408 Compare February 21, 2020 14:30
@tonyredondo tonyredondo force-pushed the no-module-package-name branch from e4df17e to 7efbb46 Compare March 2, 2020 16:31
Copy link
Contributor

@drodriguezhdez drodriguezhdez left a comment

Choose a reason for hiding this comment

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

I'd create some tests for instrumentation/testing/utils.go functions.

@tonyredondo tonyredondo force-pushed the no-module-package-name branch 2 times, most recently from 742220c to 0b37844 Compare March 14, 2020 22:49
@tonyredondo tonyredondo force-pushed the no-module-package-name branch from 927a243 to 0301130 Compare March 16, 2020 17:17
@drodriguezhdez
Copy link
Contributor

Approved

@tonyredondo tonyredondo merged commit 4648a27 into master Mar 17, 2020
@tonyredondo tonyredondo deleted the no-module-package-name branch March 17, 2020 15:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants