-
Notifications
You must be signed in to change notification settings - Fork 19
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 contributing guideline #192
Conversation
Backend code coverage report for PR #192 |
Frontend code coverage report for PR #192 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm with one fix.
Also we can add the development with local grafana section from redshift if we think it'll be useful.
1. (Optional) `mage -v buildAll` — this is optional if you need backend plugins for other platforms | ||
1. The compiled plugin should be in `dist/` directory. | ||
1. Run Grafana in [development](https://grafana.com/docs/grafana/latest/administration/configuration/#app_mode) mode, or configure Grafana to [load the unsigned plugin](https://grafana.com/docs/grafana/latest/plugins/plugin-signatures/#allow-unsigned-plugins). | ||
1. You can install by following the [install Grafana plugins docs page](https://grafana.com/docs/grafana/latest/plugins/installation/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This information will be removed by your changes. Is that intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we (this was done in a mob session last week) were trying to streamline the readme with the other plugins we have.
ea12f26
to
57b88cc
Compare
This PR adds a contributing guideline with details about how to generate the golden files.
Related to #184
Fixes #183
cc @nekketsuuu