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

configured tooltips to be defined in frontmatter and moved tooltip content to frontmatter #2192

Merged
merged 1 commit into from
Feb 13, 2021

Conversation

iesahin
Copy link
Contributor

@iesahin iesahin commented Feb 13, 2021

You may disregard these recommendations if you used the Edit on GitHub button from dvc.org to improve a doc in place.

❗ 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 choose 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. 🙏

@iesahin
Copy link
Contributor Author

iesahin commented Feb 13, 2021

I won't merge restyled results as it breaks yarn format-all

@shcheklein
Copy link
Member

@iesahin awesome! LGTM.

@shcheklein shcheklein merged commit e23d0f0 into master Feb 13, 2021
@shcheklein shcheklein deleted the iesahin/tooltips-feb-21 branch February 28, 2021 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants