I did read the wiki before doing anything and no I didn't discuss anything, maybe I should have.

If you look at junction=roundabout#Signal-controlled_roundabouts it specifically states that signal controlled roundabouts it is suggested that the roundabout tags is retained. At least one junction I did change doesn't have all sections signalled controlled so that one should definitely still be tagged a a roundabout.

In the UK everyone will call signal controlled roundabouts roundabouts.

None of the junctions has anything stated here: junction=circular#Distinguishing_from_roundabouts which definitely makes them not roundabouts.

I only changed junctions in the area I live that I have used and know about.

Having traffic lights does not stop these junctions being roundabouts. The signposts approaching them use roundabout symbols. If the traffic lights are not working they would be navigated as roundabouts. Also many of them didn't always have traffic lights when built and were roundabouts then. Adding traffic lights to a roundabout doesn't stop it being a roundabout.

These junctions were tagged as roundabouts in the past.

Changing roundabouts to circular junctions just because they have traffic lights is not a good idea.

In the UK a circular is a rare type of junction. Some do exist.

All bridges need a layer tag of 1 or higher. This is detailed in the OSM wiki on the bridge page. It doesn't matter if the bridge is a simple plank or not. It is above ground level.

In that mapillary link the bridge needs a layer tag as it crosses a river/canal. The layer is to show where a feature is in the vertical plane.

No bridge is at ground level as they cross something.

I should have put some more detail in. These layer tags were not used in conjunction with any other tags such as bridge or tunnel. The features in question could also be clearly seen at surface level on the satellite imagery and were tagged as layer -1. The layer tags were applied in error as the features should not have had any layer tag at all.

The way marked secondary link is the turn lane. I didn't modify any of the ways themselves, I just changed the highway type on the turn lane from secondary to secondary link as I thought marking turn lanes as link types is recommended in the wiki.

The changes were adding missing road names and refs.

I also spotted some roundabouts that did not have roundabout tags.

The source was looking for obvious errors on the map such as a road having a name/ref but the roundabout flares at the end of it missing that information.

Way History: 140781387 edit was to to change the roundabout flares from links to regular tertiary highways. Roundabout flares aren't classed as highway links according to the documentation.