-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
docs: migrate to github pages #3977
Conversation
thanks, big PR! |
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.
you don't need a migration to only change a help text. it would save some migrations and the need for others to keep rebasing their migrations. wdyt? (just change the existing migration(s) for this field)
Changing existing migrations usually not a good idea and can have a lot of side effects. I prefer to use our proved approach, also for small changes. |
There are no side affects if it doesn't affect the sql schema. |
I agree, we should remove these hard coded links to the documentation. |
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.
Thanks @alles-klar !
Removing the |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Conflicts have been resolved. A maintainer will review the pull request shortly. |
Migrates all links from readthedocs to github pages.