-
Notifications
You must be signed in to change notification settings - Fork 385
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
docs: rename plot -> plots #1293
Conversation
Looks like I also need to rename images and |
Getting
seems like it is not picking up that the |
Also:
should I rename these or it doesn't matter or there are some bad consequences (i dunno, maybe caching will be confused?)? |
Yeah this is a known problem in the link checker I think, please ignore it for now. Cc @casperdcl |
Good catch about the image files. I would say let's rename them. I don't think it causes any caching issue but maybe @shcheklein knows better. |
Thanks! Please request my review again if/when the core repo PR is merged. |
not problems with caching or something, just rename and change links. But to be honest it is π ... plot or plots for images names I think is not important :) |
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, please merge if/when the core repo PR is merged.
Per iterative/dvc#3802
β Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
π Please make sure to mention
Fix #issue
(if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.Please chose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. π