IIRC it was being demolished / was a pile of rubble when I passed. But I've seen fuel stations being rebuilt in months, so it wouldn't surprise me if a new one has risen already from the rubble.

I don't think "landuse=farmland" + "crop=grass" is the right tagging for meadows, especially pasture meadows. The grass is not a planted and harvested crop.

The usual tagging is `landuse=meadow` + either `meadow=pasture` or `meadow=agricultural` (hay meadow).

It is very helpful for walkers to distinguish between ploughed fields and grassland!

Fine - if Lowther Estates think of them as permissive bridleways. The only information I could find was a response to "is riding allowed?" saying that some routes were unsuitable due to cattle grids. I don't think this applies to the ways changed.

Are the trails on the Lowther Estate permissive bridleways? The mapping/information refers only to hiking and cycling:
https://lowthercastle.org/cycling-walking/estate-trails/

We can't use OS data as an information source. In any case, the more authoritative information is the council Adopted Highways data, which has the advantage of (usually) being released under an OGL licence , which is OK for OSM.

Specifically I'm reverting Firby Lane in Ripon as this is a residential/unclassified road and not a service road.

Thanks! I've retagged as was:tourism etc. so make this clear.

Has Bishopton Grove House been converted to apartments from a B7B? I was adding a B&B based on an FHRS ID, but this may be out-of-date.

Sorry to be picky, but `surface=paving_stones` is the commonly used tagging rather than `surface=flagstones`. "paving_stones" is one of the recognised "paved surface types" in the "standard" map.

For the Boots in Team Valley, it would probably be better to have a single `amenity=pharmacy` with `opening_hours:pharmacy`?
It will be confusing for data consumers to have two `amenity=pharmacy`.

Is level=1 correct for this ATM? It would imply it was on an upper floor, but the building looks to be a single level.

I think re-tagging the footways in the Cornmill centre from highway=footway to highway=corridor makes the map less usable - it it hard to understand the connectivity.

`highway=corridor` might make sense if mapping the corridors of a complex building, but is really part of a separate tagging scheme (simple indoor mapping).

Have these PRoWs been confirmed independently e.g. for Strava data? I'm looking at the N/S footway on Hedleyhope Fell and can't see how it relates to ground features.

I've corrected this to `amenity=cafe`. `shop=coffee` is literally a shop that sells coffee beans etc.

Is "Brook Lane Court" correct for the overall collection of buildings? The individual buildings are Broad Lane Court.

Good catch! Embarrassing as I originally added it...

It was confusing because the listing is "Barclays Bank", but refers to the building itself, not the bank brand.

Fixed.

I'm not sure about the name change here:
osm.org/edit#map=19/54.518700/-1.502995
from
"Morton Park Beefeater"
to
"Morton Park"

The latter is the branch name used on the website, but the restaurant itself would be identified as the "Morton Park Beefeater". "Morton Park" is the name of the area, and so it is a bit confusing for the two to have the same name. The visible signage has just "Beefeater".

If the car park is open, why not tag this as amenity=parking?

That's not what permissive means. The original tagging was:
motor_vehicle = no
motor_vehicle:conditional = yes @ (19:00-7:00)
which encodes the 7-7 restriction correctly.

I will revert for now, but it probably needs further checking if the sign has changed.

Conditional access is a very advanced topic and one that has big effects on routers etc. Best to steer clear or ask unless you are 100% sure.

Please also respond to my separate changeset query about not:brand changes.

"motor_vehicle=permissive" only makes sense on a private-owned highway, whereas this is a public highway. Access restrictions are complex, so I think you need to explain how exactly the sign has changed.

I'm not clear why wikidata:brand = Q37158 was changed to
not:wikidata:brand
This is the brand:wikidata used to identify Starbucks locations, for example, in this tool:
https://osm.mathmos.net/chains/Q37158/