-
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
Kinetica exporter/pr 2 metrics - updated orderedmap version to v2 #30398
Kinetica exporter/pr 2 metrics - updated orderedmap version to v2 #30398
Conversation
…m-kinetica/opentelemetry-collector-contrib into kinetica-exporter/PR-2-metrics
…m-kinetica/opentelemetry-collector-contrib into kinetica-exporter/PR-2-metrics
…m-kinetica/opentelemetry-collector-contrib into kinetica-exporter/PR-2-metrics
…m-kinetica/opentelemetry-collector-contrib into kinetica-exporter/PR-2-metrics
hey, this PR is out of wack. Let me see if I can help.
Then create a new branch
Now merge squash your changes into it:
Commit the changes as you normally would.
Check if that looks good on this PR.
|
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.
We can exclude the changelog since it is a dep bump
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.
How did this even get changed ? I didn't add anything to this file.
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.
Probably something messed up in your branch. We delete changelog files after a release and since we did a release on Tuesday this file no longer exists in main so your branch is adding it.
…en-telemetry#30398) **Description:** Updated orderedmap version to v2 **Link to tracking Issue:** 24294 **Testing:** Existing tests were run **Documentation:** None
Description: Updated orderedmap version to v2
Link to tracking Issue: 24294
Testing: Existing tests were run
Documentation: None