Skip to content

Update subway operator details to Macao Light Rapid Transit - #12169

Open
qgmzmy wants to merge 3 commits into
osmlab:mainfrom
qgmzmy:patch-1
Open

Update subway operator details to Macao Light Rapid Transit#12169
qgmzmy wants to merge 3 commits into
osmlab:mainfrom
qgmzmy:patch-1

Conversation

@qgmzmy

@qgmzmy qgmzmy commented May 7, 2026

Copy link
Copy Markdown

No description provided.

@UKChris-osm UKChris-osm added the add transit Actionable - add a transit network to the index label May 10, 2026
@Vectorial1024

Copy link
Copy Markdown
Contributor

The Wikidata tag should probably also be updated (if exists).

Updated the Wikidata operator ID for the Macao Light Rapid Transit.
@bhousel

bhousel commented Jul 27, 2026

Copy link
Copy Markdown
Member

We generally don't include corporate forms like "Limited" and "S.A." in the tag values. There have even been concerted efforts to remove these from OSM in the past.

If there is a strong community preference to include them, then sure, but most communities prefer not to.
Just wanted to ask "Are you really sure?" before I merge.

@Vectorial1024

Copy link
Copy Markdown
Contributor

To clarify, I am only here because I scrolled by this and noticed the PR back then did not update the Wikidata field.

This Macao Light Rail started as a subsidiary of HK MTR, and I would agree/prefer to remove the "Limited" from MTR even before the change of operators.

@qgmzmy

qgmzmy commented Jul 30, 2026

Copy link
Copy Markdown
Author

We generally don't include corporate forms like "Limited" and "S.A." in the tag values. There have even been concerted efforts to remove these from OSM in the past.

If there is a strong community preference to include them, then sure, but most communities prefer not to. Just wanted to ask "Are you really sure?" before I merge.

Not really sure either, I was just following what the wiki seemed to suggest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add transit Actionable - add a transit network to the index

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants