The Wayback Machine - https://web.archive.org/web/20240110183215/https://github.com/matplotlib/matplotlib/pull/27560
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

Moved /users/project to /doc/project #27560

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

shriyakalakata
Copy link
Contributor

PR summary

This PR addresses the issue #27396 on moving non user guide related content out of /users. This is the first step, moving /users/project -> /doc/project and adding redirects.

PR checklist

@story645
Copy link
Member

Thanks for taking this on!
This PR needs to be rebased against the current version of main:
https://matplotlib.org/devdocs/devel/development_workflow.html#rebase-onto-upstream-main

And I'm pretty sure the paths of all the license files need to be updated now that they've been moved up a level.

@story645
Copy link
Member

Awesome, am waiting on doc build to check all the links.
Can you please also update https://github.com/matplotlib/matplotlib/blob/main/doc/devel/contribute.rst#contribute-documentation ?

@story645
Copy link
Member

story645 commented Jan 10, 2024

Awesome that you caught the pre-commit. Doing a small git grep users/project these were the other paths/urls that should be changed: (urls should still work b/c of redirects but may as well do the updates together).
(mpl-dev) C:\Users\story\Projects\matplotlib>git grep "users/project"

And the docs readme that's already slightly out of date

@story645
Copy link
Member

Also let me know if you'd like to do the updates to the brochure site or if you'd like me to. Again, b/c of the redirects the urls should still work, so it shouldn't be urgent:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Doc]: Move non user guide related content out of /users
2 participants