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

Update readme with newer validator and docker command information #218

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

academo
Copy link
Member

@academo academo commented Jun 18, 2024

closes #170

@academo academo requested review from a team as code owners June 18, 2024 11:55
@academo academo requested review from mckn, marefr, andresmgot and xnyo and removed request for a team June 18, 2024 11:55
| Developer Jargon / `jargon` | Generally discourages use of code jargon in the documentation. | None |
| JS Source Map / `jssourcemap` | Checks for required `module.js.map` file(s) in archive. | `sourceCodeUri` |
| Legacy Grafana Toolkit usage / `legacybuilder` | Detects the usage of the not longer supported Grafana Toolkit. | None |
| Nested includes metadata / `includesnested` | Validates that nested plugins have the correct metadata. | |
Copy link
Member Author

Choose a reason for hiding this comment

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

This line is new. All the others modification are auto-formatting shifting because of the longer content

@academo academo requested review from tolzhabayev and sympatheticmoose and removed request for mckn, marefr, andresmgot and xnyo June 18, 2024 11:56
Copy link
Collaborator

@briangann briangann left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Member

@xnyo xnyo left a comment

Choose a reason for hiding this comment

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

LGTM!

@academo academo merged commit bd40d9f into main Jun 24, 2024
3 checks passed
@academo academo deleted the academo/update-readme-commands branch June 24, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Document checksum parameter
3 participants