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

Move GTO docs here from mlem.ai #4822

Merged
merged 4 commits into from
Sep 1, 2023
Merged

Move GTO docs here from mlem.ai #4822

merged 4 commits into from
Sep 1, 2023

Conversation

aguschin
Copy link
Contributor

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

@aguschin aguschin requested a review from a team as a code owner August 31, 2023 10:42
@aguschin aguschin self-assigned this Aug 31, 2023
@aguschin aguschin added the A: docs Area: user documentation (gatsby-theme-iterative) label Aug 31, 2023
@shcheklein shcheklein temporarily deployed to dvc-org-add-gto-5vgwimjvp7wbf5 August 31, 2023 10:45 Inactive
@aguschin aguschin requested a review from omesser August 31, 2023 10:54
@github-actions
Copy link
Contributor

github-actions bot commented Aug 31, 2023

Link Check Report

There were no links to check!

@shcheklein
Copy link
Member

@aguschin there are some failing links, could you please take a look?

@dberenbaum
Copy link
Contributor

@aguschin there are some failing links, could you please take a look?

Looks like they are links to mlem. Do we want to drop mlem references here or at least note the project status?

Otherwise LGTM!

@shcheklein
Copy link
Member

I think it's better to drop them.

@shcheklein shcheklein temporarily deployed to dvc-org-add-gto-5vgwimjvp7wbf5 September 1, 2023 13:50 Inactive
@aguschin
Copy link
Contributor Author

aguschin commented Sep 1, 2023

Fixed links + dropped MLEM references. There is 1 broken link, it will be fixed by itself after this PR is merged.

@aguschin
Copy link
Contributor Author

aguschin commented Sep 1, 2023

Looks like it can be merged :)

@shcheklein shcheklein merged commit 73bce70 into main Sep 1, 2023
4 checks passed
@shcheklein shcheklein deleted the add-gto branch September 1, 2023 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: docs Area: user documentation (gatsby-theme-iterative)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants