Great.  A few other random thoughts I noted while I think of them:

Try to keep your changeset comments specific, so that you and others can see the basic content and reason for the change; as a rough guide you should be able to look down your list of changes and fairly easily remember which was which and others should be able to see whether they are interested in them. Most of yours are great, but avoid "small adjustment" or similar generic phrases.

We like to keep the history (of ways, and single tagged nodes - but ordinary untagged nodes of ways aren't valuable though) so if you rework a building, road or even a shop node that's already mapped, try to keep the old way/single node and change the tags eg shop to vacant or whatever.  Obviously if there's demolition and rebuilding something else, then delete and re-add as necessary.

A couple of times you've made a building of parts by having untagged closed areas grouped by a building relation.  This is actually not usual, I made the areas into building:part areas and drew an outline around the whole as the building way.  ('building:part' can be used to do 3d buildings which is a whole other area you can look into some time if you fancy it).  Multipolygons are mostly used where there is a hole inside, I put one at one of the school buildings where there's grass in the middle.

You took out the pedestrianised 'linear' way at Spring Gardens (and some connecting side ways).  I've put these back, as OSM is not just one (or more) renders of a map like you see on the OSM website, its really a database of connected ways (etc), and for calculating routes and other purposes the linear ways are more significant.  Also there was a cycle route relation (an ordered list of ways representing a named followable route) down Spring Gardens which got broken, but all is good now.

Have fun mapping,
Cebderby (Clive)