Skip to content

feat: Support publishing plugins with team and kind metadata set#1313

Merged
hermanschaaf merged 8 commits intomainfrom
publish-plugins
Oct 19, 2023
Merged

feat: Support publishing plugins with team and kind metadata set#1313
hermanschaaf merged 8 commits intomainfrom
publish-plugins

Conversation

@hermanschaaf
Copy link
Contributor

This allows us to package the plugin with team and kind metadata set, needed for publishing to the new registry and checking usage and limits against the new cloud API.

The package command no longer needs to take a kind argument.

Also adds an info command to print this information, useful for debugging, otherwise it won't be visible anywhere. We can consider adding this to the --version output instead, but that would technically be breaking?

"github.com/cloudquery/plugin-sdk/v4/plugin"
)

// These variables are set as part of the `package` command
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this comment correct now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh no, thanks

@github-actions
Copy link

github-actions bot commented Oct 18, 2023

⏱️ Benchmark results

Comparing with 16e36d6

  • Glob-8 ns/op: 90.96 ⬆️ 0.59% increase vs. 16e36d6

@hermanschaaf
Copy link
Contributor Author

I'll merge this soon. After it rolls out, we'll need to update all the source plugins that are published to the new registry to set these new properties.

@hermanschaaf hermanschaaf merged commit 933698d into main Oct 19, 2023
@hermanschaaf hermanschaaf deleted the publish-plugins branch October 19, 2023 08:55
hermanschaaf pushed a commit that referenced this pull request Oct 19, 2023
🤖 I have created a release *beep* *boop*
---


##
[4.16.0](v4.15.3...v4.16.0)
(2023-10-19)


### Features

* Support publishing plugins with team and kind metadata set
([#1313](#1313))
([933698d](933698d))

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