-
Notifications
You must be signed in to change notification settings - Fork 440
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
internal: add supported versions for v2 #3070
base: v2-dev
Are you sure you want to change the base?
Conversation
Datadog ReportBranch report: ✅ 0 Failed, 4084 Passed, 64 Skipped, 2m 49.72s Total Time |
# note: This will only run when there *are* changes to integration versions | ||
- name: Create Pull Request | ||
id: pr | ||
uses: peter-evans/create-pull-request@v6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BenchmarksBenchmark execution time: 2025-01-15 17:58:09 Comparing candidate commit 8ce43a8 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 51 metrics, 2 unstable metrics. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work! this is looking pretty good :D
Left some minor comments / potential improvements
@@ -844,3 +844,11 @@ func isAWSMessagingSendOp(awsService, awsOperation string) bool { | |||
} | |||
return false | |||
} | |||
|
|||
func GetPackages() map[Package]string { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@darccio @rarguelloF what do you think? I changed this to return a deep copy, and only store the values we need to use
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me, but in case you want the other public fields as well, you can use the json.Marshal
/ json.Unmarshal
trick to do it easily (it's not performant at all, but for this use case it should be good enough)
What does this PR do?
contrib/<package_name>/go.mod
, and the maximum version from github.- This fetches the maximum from
go list -m -u -json <repository
contrib/supported_versions.md
with the module name, repository, minimum and maximum versions supported, and whether it is auto-instrumented using this definition: https://github.com/DataDog/orchestrion#supported-librariesMotivation
Reviewer's Checklist
v2-dev
branch and reviewed by @DataDog/apm-go.Unsure? Have a question? Request a review!