-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[chore][internal/k8stest] Make k8stest package publicly available #36661
base: main
Are you sure you want to change the base?
Conversation
02995d2
to
e822916
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.
Looks good on the Datadog side.
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.
I'd like to continue discussing this proposal in the issue. Making things public puts a burden on maintaining.
Sure, let's discuss. I opened this PR just to visualize what I am proposing. |
8df0b90
to
b0cb0cd
Compare
0c31374
to
43cf2de
Compare
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
not stale |
fc687df
to
129c5e9
Compare
129c5e9
to
66fee7a
Compare
unblocking based on discussion in issue
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
66fee7a
to
74dc5fe
Compare
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
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.
Can we note in the readme that this package is experimental and may break at any time without notice?
Description
Link to tracking issue
Part of #36615