-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Update links to new documentation #38
Conversation
This is related to PR2257 of sopel-irc/sopel repo. See also: sopel-irc/sopel#2257
Is this meant to add redirects from the old to the new paths? I don't see those changes. I also have to make extra sure that I don't merge this before the docs are updated. So if you don't mind, I'll convert it to a draft. 😁 |
Oh right, I forgot about redirect! |
Redirects done. I think it should be enough. I'm not sure, it's always a bit hard to track everything: I looked at the files I moved, not the newly created. |
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, though I will have to hold off on actually merging this until the new Sopel version is out. Expect the base branch of this PR to be changed later, when there is a "Sopel 8" branch to merge it into with other updates (new migration guide etc.).
As expected! |
Going to put all this into #47 after updating some of the underlying config/tooling (like the outdated Python version Netlify was configured to build with when this PR started). |
…or I'll forget to change the base and merge to |
This is related to PR 2257 of sopel-irc/sopel repo.
I tried to find all the possible links, so I may have miss some. The main two changes being:
I also added a few links because I think they are appropriate and in-line with a more user-friendly approach to documentation.