I'm not "telling you what to do". It's a notification that you can ignore if you wish. Don't complain when I change it back that is.

Please don't add descriptor labels to `name=` . It is for proper names only. Use `description:en=` + `description:zh=` .

`bridge=covered` could be used. It means fully enclosed. The namesake type can be indicated with `bridge:material=wood` . Eg `=viaduct` doesn't have to be steel or concrete. As long as it has many spans.
`covered=yes` doesn't have this meaning. It's not very `indoor=yes` either.

Yes there is. `substance=` is widely used and supported. Using `pumping_station=` prevents it from being used for other purposes.
You are the one changing it meaninglessly. I'm telling you why not to. There's no "rules". Doing something "wrong" is not the only reason for receiving comments.
Wiki is not for personal opinions either.
And you haven't considered `pipeline=substation`.

I haven't added `pipeline=substation` . `substance=` is used with it.

Please use `abandoned:highway=path` to avoid others re-adding it again. It's still shown on other maps.

Originally it is worded as "It is better to use the name of the substance transported with the key substance=*" osm.wiki/w/index.php?title=Tag:man_made=pumping_station&oldid=1881857

`substance=` is general standard used for all features, and "approved" formally. There are only 3.5k `pumping_station=` .

The wiki is not a bible. It's obviously self-conflicting to say those are "The activity in which the station is involved". What is the `=water` or `=gas` used for? Drinking and cooking, electricity generation, or heating? "The actual substance processed" is a better definition, and that's using `substance=` already.

Please don't change `substance=` to `pumping_station=` . This is a meaningless duplicate. man_made=pumping_station#Tags_used_in_combination

`landuse=` is about the use. Not zoning or laws.

`=headrace` is used for hydropower only. It's not applicable.

Please don't delete `landuse=residential` . They are valid data.

As I have said before, please don't change the `name=` to compose housing estate + building name. This is an application issue.

We touched upon this issue now changeset/142112318

Please don't remove the `addr:street=` . They are done for a specific purpose for HK. Street naming usually applies until the gore, so the section is actually Lion Rock Tunnel Rd. But logically and routing-wise, it should be considered Shatin Rd. However, using `official_name=` is unclear.

Eg Ma Chai Hang Rd rounabout's street naming is Ma Chai Hang Rd.

Street naming is inconsistent. See Fung Mo Street.
It can also be incomplete. Many road sections don't have a street name gazetted or signposted.
The incompleteness and inconsistency can play together when addressing doesn't follow the absence of street naming.
Another clear example is how `=roundabout` is described to not have names internationally unless it is the junction name itself. But HK applies the road name to all roundabouts.

No, don't say you are copying from Google. This is not allowed. It's a copyright infringement.
I don't agree. Street names don't decide anything. In fact, `*_link=` is described to not have names internationally, which is not applicable. In Hong Kong, the street naming have a very specific organization that defies routing expectation. For example, on slip roads, the name of the main line applies until the gore. Then routing will be confused unless it evaluates the name afterwards.

It's not about the physical design, or whether it is an "exit" or "entrance" . Contrary to Google Maps, which has many other arguably bad designs.