-
-
Notifications
You must be signed in to change notification settings - Fork 299
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
g.download.project: rename g.download.location #4187
Conversation
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.
Looks good, assuming that the old contents of g.download.location were copied to g.download.project without hidden changes. The some docstrings could be changed later (more reviewable diff) to change location->project when needed.
The old g.download.location calls the new g.download.project, looks fine.
Html docs are also updated.
Please merge #4155 first. |
Is #4155 almost ready, or ready? |
@petrasovaa there is a conflict, but the change(s) from #4155 might need to be applied to a different file. It needs more than what the web interface offers to do it right. |
04071df
to
79db0bc
Compare
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.
Let's go with this. Thanks Anna!
@petrasovaa Do you want this to be backported to G8.4? For now it landed in G8.5. |
Right, done. |
To continue with the location to project renaming, this PR is renaming g.download.location to g.download.project. For backwards compatibility I left g.download.location that calls g.download.project.
My suggestion is to add a warning to g.download.location in/after 8.5 and remove it completely for GRASS 9.