docs: Replace paver update_db. #1799
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We are planning on removing paver, and we have created an alternative Make command for running migrations. This updates the documentation. openedx-unsupported/devstack#1085
robrap
approved these changes
Jul 12, 2023
| @@ -146,7 +146,7 @@ as an example) and run migrations: | |||
|
|
|||
| .. code-block:: bash | |||
|
|
|||
| $ paver update_db | |||
| $ make migrate | |||
| # Or use a more down-to-the-root command (replace aws with your version of config) | |||
There was a problem hiding this comment.
Not part of your PR, but what is meant by aws here?
There was a problem hiding this comment.
Also, I just remembered that the old call had paver install_prereqs baked in. Does that need to be added here? Also, do we have a Make equivalent for that, or is that something needed later?
There was a problem hiding this comment.
We have make requirements, but I don't think this documentation requires the requirements to be installed. (aws is the old name of the production settings, I believe.)
There was a problem hiding this comment.
- This comment doesn't make sense in this file, because we're using
devstack, and it probably doesn't need to be replaced? - In the other file, it probably makes sense to replace
awsin the comment and the example command toproduction.
Since this merged already, I guess we'll see if one of us do a follow-up PR. :)
macdiesel
approved these changes
Jul 13, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We are planning on removing paver, and
we have created an alternative Make command for
running migrations. This updates the documentation.
openedx-unsupported/devstack#1085
I haven't bumped the version or added a changelog entry because this is a minor docs change.