Skip to content

fix: Rename plugin type -> kind for consistency with existing configs#1240

Merged
kodiakhq[bot] merged 1 commit intomainfrom
kind
Sep 21, 2023
Merged

fix: Rename plugin type -> kind for consistency with existing configs#1240
kodiakhq[bot] merged 1 commit intomainfrom
kind

Conversation

@hermanschaaf
Copy link
Copy Markdown
Contributor

I realized existing plugin configs use the word kind (e.g. kind: source) and not type. Let's just stick to one word everywhere for consistency.

@github-actions
Copy link
Copy Markdown

⏱️ Benchmark results

Comparing with dac3376

  • Glob-8 ns/op: 99.59 ⬆️ 0.45% increase vs. dac3376

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 21, 2023

Codecov Report

Patch coverage: 40.00% and no project coverage change.

Comparison is base (dac3376) 48.80% compared to head (c984f20) 48.80%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1240   +/-   ##
=======================================
  Coverage   48.80%   48.80%           
=======================================
  Files          89       89           
  Lines        8310     8310           
=======================================
  Hits         4056     4056           
  Misses       3877     3877           
  Partials      377      377           
Files Changed Coverage Δ
plugin/plugin_package.go 0.00% <0.00%> (ø)
serve/package.go 57.75% <66.66%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@disq
Copy link
Copy Markdown
Member

disq commented Sep 21, 2023

Good catch!

@kodiakhq kodiakhq bot merged commit a00b8d0 into main Sep 21, 2023
@kodiakhq kodiakhq bot deleted the kind branch September 21, 2023 14:22
kodiakhq bot pushed a commit that referenced this pull request Sep 21, 2023
🤖 I have created a release *beep* *boop*
---


## [4.10.2](v4.10.1...v4.10.2) (2023-09-21)


### Bug Fixes

* Rename plugin type -&gt; kind for consistency with existing configs ([#1240](#1240)) ([a00b8d0](a00b8d0))

---
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.

3 participants