Skip to content

feat: add plugin-directory input to e2e-version - #212

Draft
sunker wants to merge 4 commits into
mainfrom
feat/plugin-path-input
Draft

feat: add plugin-directory input to e2e-version#212
sunker wants to merge 4 commits into
mainfrom
feat/plugin-path-input

Conversation

@sunker

@sunker sunker commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a plugin-directory input to the e2e-version action so it can find src/plugin.json in the correct location for plugins that live in a subdirectory (e.g. monorepos).

Without this, the action always reads grafanaDependency from the repo root's src/plugin.json, ignoring the actual plugin location. The only workaround was to manually set grafana-dependency, which is easy to forget and gets stale.

Changes

  • action.yml: add optional plugin-directory input
  • index.js: read plugin.json from plugin-directory when set
  • README.md: document the new input

Usage

- uses: grafana/plugin-actions/e2e-version@e2e-version/vX.Y.Z
  with:
    version-resolver-type: plugin-grafana-dependency
    plugin-directory: path/to/plugin

@sunker sunker changed the title (feat): add plugin-path input to e2e-versions feat: add plugin-path input to e2e-versions Apr 15, 2026
@sunker sunker self-assigned this Apr 15, 2026
@sunker sunker moved this from 📬 Triage to 🧑‍💻 In development in Grafana Catalog Team Apr 15, 2026
@sunker
sunker force-pushed the feat/plugin-path-input branch from c81db32 to 0939226 Compare April 24, 2026 11:42
@sunker sunker changed the title feat: add plugin-path input to e2e-versions feat: add plugin-directory input to e2e-version Apr 27, 2026
@sunker sunker moved this from 🧑‍💻 In development to 🔬 In review in Grafana Catalog Team May 13, 2026
@tolzhabayev

Copy link
Copy Markdown
Contributor

@sunker obsolete?

@sunker

sunker commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

I guess - waiting for response here: grafana/plugin-ci-workflows#696 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🔬 In review

Development

Successfully merging this pull request may close these issues.

2 participants