Skip to content
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

Add bundle plugin #71

Closed
wants to merge 39 commits into from
Closed

Conversation

SandersAaronD
Copy link

@SandersAaronD SandersAaronD commented Mar 22, 2024

This bundles a grafana plugin, which can have a backend or no backend, into a properly signed folder of zip files for both current version and "latest" which can be uploaded in order to publish the plugin. By default it zips from ./dist into __to-upload__, but both of those are configurable parameters.

This workflow is structured the way it is with most logic in a sub-package to facilitate using the package as a standalone command outside of a github action if someone has a workflow that calls for that or is using this logic somewhere outside of Grafana to package a plugin.

Implemented in typescript for good portability and maintainability and to integrate well with github actions without introducing any extra dependencies (on, e.g., golang runtimes or binaries). The same thing can presently be accomplished by downloading a grabpl binary onto a machine with a golang runtime and calling it, as can be seen here.

@SandersAaronD SandersAaronD marked this pull request as ready for review March 27, 2024 13:39
@SandersAaronD SandersAaronD requested a review from a team as a code owner March 27, 2024 13:40
@dsotirakis
Copy link
Contributor

Hey @SandersAaronD 👋
Do you have any updates on this about getting it moved from this repo?

@guicaulada
Copy link
Contributor

Still waiting on update from @SandersAaronD

@iainlane
Copy link
Member

iainlane commented Sep 25, 2024

Hey folks, we've had discussions here and in other channels that this PR is considered too specialised to be included here.

As a result of this PR we documented what the scope of this repository is, which was something we lacked before.

I'm going to close this PR. I encourage you @SandersAaronD to find a home closer to the repos that will need this.

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.

4 participants