Skip to content

Conversation

@karthikscale3
Copy link
Contributor

@karthikscale3 karthikscale3 commented Oct 23, 2024

Another incremental minor update for the sake of better clarity. Shown below the pre post of the changes. Happy to revert back anything you feel is unnecessary. Also, these changes DO NOT affect the permalinks so it won't cause any breakages to existing links on the internet.

This PR also removes the duplicate assertions.md file inside modules as the same doc is available at the top level under concepts.

image

@krypticmouse
Copy link
Collaborator

krypticmouse commented Oct 23, 2024

It looks good for the most part but this might break google index in the short term, so I suggest to add some redirection links in mkdocs

Check #1681 for reference

Thank you so much for the contribution!!

@karthikscale3
Copy link
Contributor Author

It looks good for the most part but this might break google index in the short term, so I suggest to add some redirection links in mkdocs

Check #1681 for reference

Thank you so much for the contribution!!

Good call. This PR does not actually change the links. I verified it and it keeps the old links. I just added one redirect for the assertions doc inside deep-dive section which was duplicated so I removed it and added a redirect for it.

@krypticmouse
Copy link
Collaborator

Awesome! LGTM 🚢

Thank you so much for the contribution!!

@krypticmouse krypticmouse merged commit 56dec59 into stanfordnlp:main Oct 23, 2024
6 checks passed
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