-
Notifications
You must be signed in to change notification settings - Fork 154
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
Fix some minor cosmetic issues in docs #2218
Conversation
Let's update varialbes.md to use the proper <varialbe> syntax instead of the legacy <span> syntax for defining variables.
Let's remove the variables.json file as it is not used.
Let's update links of example dashboards to point to a later version which uses a more recent version of RepoSense.
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, thanks!
I'll have to squash merge as "Rebase and merge" is disabled for RepoSense. |
@ckcherry23 I enabled it just now. |
Completed in #2221 to rebase with master instead of merge commits |
The following links are for previewing this pull request:
|
Fixes for some minor cosmetic issues I noticed while reading the UG.
You can do a rebase merge for this, as each commit is a separate fix.