-
Notifications
You must be signed in to change notification settings - Fork 197
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
adding telemetry #1692
base: branch-24.12
Are you sure you want to change the base?
adding telemetry #1692
Conversation
7d52daa
to
2b4f54b
Compare
953f9b8
to
6e97890
Compare
8a76653
to
7a6f605
Compare
0674d3e
to
4caf981
Compare
8410ebb
to
8d5473a
Compare
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.
What can we do to make this less verbose?
ref: ${{env.SHARED_ACTIONS_REF}} | ||
path: ./shared-actions | ||
- id: traceparent | ||
uses: ./shared-actions/telemetry-traceparent |
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.
Must we clone this repository? Can we not use a "shared action" as usual?
Something like uses: rapidsai/shared-actions/...@main
?
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.
Yes, we must clone it. There's an explanation for this change at https://github.com/rapidsai/shared-actions. TL;DR - this is the only way to allow shared actions to depend on other shared actions.
Co-authored-by: Bradley Dice <[email protected]>
Description
close #1691
Checklist