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

New analyzer: detect usage of go-sdk and version for backend plugins #126

Closed
sympatheticmoose opened this issue May 17, 2023 · 2 comments · Fixed by #198
Closed

New analyzer: detect usage of go-sdk and version for backend plugins #126

sympatheticmoose opened this issue May 17, 2023 · 2 comments · Fixed by #198
Assignees
Labels
enhancement New feature or request

Comments

@sympatheticmoose
Copy link
Contributor

sympatheticmoose commented May 17, 2023

Provide a new analyzer to detect whether the go-sdk is being used for a backend component and which version is being used.

It could check for backend=true in the plugin.json file. If yes, scan for a go.mod file in the project and check the version declared there.

To start with, it should warn if the sdk is not being used and if the version is not latest (as well as printing "we recommended updating to the current version ".

In the future we can consider failing if the sdk is not being used and if the version is significantly out of date

@sympatheticmoose sympatheticmoose added the enhancement New feature or request label May 17, 2023
@academo academo self-assigned this May 2, 2024
@academo
Copy link
Member

academo commented May 6, 2024

Solved with #192

@academo academo closed this as completed May 6, 2024
@sympatheticmoose
Copy link
Contributor Author

@academo the linked PR doesn't seem to address the version and hasn't been added to the readme https://github.com/grafana/plugin-validator?tab=readme-ov-file#analyzers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants