Skip to content

contrib/emicklei/go-restful.v3: fix componentName#2132

Merged
knusbaum merged 2 commits into
mainfrom
rarguelloF/go-restful-component
Jul 20, 2023
Merged

contrib/emicklei/go-restful.v3: fix componentName#2132
knusbaum merged 2 commits into
mainfrom
rarguelloF/go-restful-component

Conversation

@rarguelloF

@rarguelloF rarguelloF commented Jul 20, 2023

Copy link
Copy Markdown
Contributor

What does this PR do?

After #2092, where the integration was moved to the correct folder, I forgot to update the componentName as well.

Motivation

Describe how to test/QA your changes

Reviewer's Checklist

  • Changed code has unit tests for its functionality.
  • If this interacts with the agent in a new way, a system test has been added.

@rarguelloF
rarguelloF force-pushed the rarguelloF/go-restful-component branch from 9526d70 to edae318 Compare July 20, 2023 09:52
@rarguelloF
rarguelloF marked this pull request as ready for review July 20, 2023 09:53
@rarguelloF
rarguelloF requested a review from a team July 20, 2023 09:53
@pr-commenter

pr-commenter Bot commented Jul 20, 2023

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2023-07-20 14:31:53

Comparing candidate commit ad199c1 in PR branch rarguelloF/go-restful-component with baseline commit cf96d3a in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 27 metrics, 0 unstable metrics.

@knusbaum
knusbaum merged commit 21ccc69 into main Jul 20, 2023
@knusbaum
knusbaum deleted the rarguelloF/go-restful-component branch July 20, 2023 14:32
knusbaum pushed a commit that referenced this pull request Jul 20, 2023
After #2092, where the integration was moved to the correct folder, I forgot to update the componentName as well.
knusbaum added a commit that referenced this pull request Jul 20, 2023
…#2136)

After #2092, where the integration was moved to the correct folder, I forgot to update the componentName as well.

This is a cherry-pick of #2132 onto the v1.53.x release branch.
katiehockman pushed a commit that referenced this pull request Jul 26, 2023
After #2092, where the integration was moved to the correct folder, I forgot to update the componentName as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants