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.

Sign up for GitHub

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

Jump to bottom

v.info: Rename location to project in outputs #3133

Merged
merged 2 commits into from
Mar 22, 2024
Merged

v.info: Rename location to project in outputs #3133

merged 2 commits into from
Mar 22, 2024

Conversation

wenzeslaus
Copy link
Member

  • Translatable plain text output now generates "Project" instead of "Location".
  • Extended shell-style output now generates "project=" in addtion to "location=".
  • Examples in documentation now show project only.
  • History continues using LOCATION because history file is printed as is and the parsing rules applied to it are unclear, so consequences of adding a new key there are not clear to me.

@wenzeslaus
v.info: Rename location to project in outputs
1a76f8f 
- Translatable plain text output now generates "Project" instead of "Location".
- Extended shell-style output now generates "project=" in addtion to "location=".
- Examples in documentation now show project only.
- History continues using LOCATION because history file is printed as is and the parsing rules applied to it are unclear, so consequences of adding a new key there are not clear to me.
@wenzeslaus wenzeslaus added enhancement New feature or request C Related code is in C labels Aug 25, 2023
@wenzeslaus wenzeslaus added this to the 8.4.0 milestone Aug 25, 2023
@github-actions github-actions bot added vector Related to vector data processing HTML Related code is in HTML module docs labels Feb 14, 2024
cmbarton
cmbarton approved these changes Feb 14, 2024
View reviewed changes
Copy link
Contributor

@cmbarton cmbarton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes! I agree with this standardized usage and recommend it for all other places where these terms are used.

@petrasovaa petrasovaa merged commit 553f861 into OSGeo:main Mar 22, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cmbarton cmbarton cmbarton approved these changes

@petrasovaa petrasovaa petrasovaa approved these changes

Assignees
No one assigned
Labels
C Related code is in C docs enhancement New feature or request HTML Related code is in HTML module vector Related to vector data processing
Projects
Rename location to project in GRASS GIS
Status: Done
Milestone
8.4.0
Development

Successfully merging this pull request may close these issues.
3 participants
@wenzeslaus @petrasovaa @cmbarton