Skip to content

fix(destination): Rename NewDestinationPlugin to NewPlugin#519

Merged
kodiakhq[bot] merged 1 commit intocloudquery:mainfrom
erezrokah:fix/rename_destination_new_method
Dec 19, 2022
Merged

fix(destination): Rename NewDestinationPlugin to NewPlugin#519
kodiakhq[bot] merged 1 commit intocloudquery:mainfrom
erezrokah:fix/rename_destination_new_method

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

Follow up to #516


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

@github-actions
Copy link
Copy Markdown

⏱️ Benchmark results

Comparing with 74c6672

  • DefaultConcurrency-2 resources/s: 12,017 ⬆️ 0.97% increase vs. 74c6672
  • Glob-2 ns/op: 187.2 ⬆️ 1.12% increase vs. 74c6672
  • TablesWithChildrenDefaultConcurrency-2 resources/s: 28,511 ⬆️ 0.31% increase vs. 74c6672
  • BufferedScanner-2 ns/op: 11.79 ⬇️ 0.85% decrease vs. 74c6672
  • LogReader-2 ns/op: 34.74 ⬇️ 2.48% decrease vs. 74c6672

Copy link
Copy Markdown
Contributor

@yevgenypats yevgenypats left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. I already did it in my Follow-up PR but I get that otherwise we will need to fix breaking compilation changes twice so better to do it once now.

@kodiakhq kodiakhq bot merged commit 3934775 into cloudquery:main Dec 19, 2022
@erezrokah erezrokah deleted the fix/rename_destination_new_method branch December 19, 2022 11:44
kodiakhq bot pushed a commit that referenced this pull request Dec 19, 2022
🤖 I have created a release *beep* *boop*
---


## [1.12.7](v1.12.6...v1.12.7) (2022-12-19)


### Bug Fixes

* **destination:** Rename `NewDestinationPlugin` to `NewPlugin` ([#519](#519)) ([3934775](3934775))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants