hi I updated the bounds per the LPI Basemap it looks like you included an apartment building as part of the school, if you have local knowledge if you could confirm that? I also updated the tags of the buildings.

Hi Stephen, this is good, but a few tips since you asked for a review.
1. looks like you have a few short way segments these could probably be joined together into one way unless you need to use different tags for each section.
2. unless the cycleway is closed, i would leave out access=no, although the bicycle=yes would allow bicycles, access=no could be misinterpreted so it's better to not include that in my opinion.
3. if the pedestrians and cyclists share the same space and don't have their own lanes you can add segregated=no

golf=tee is the common tag for tee's so I've changed the tag.

hi you deleted the water areas here, could you please elaborate on why?

If it's a declared Wildlife Protection Area by the local council you can consider adding boundary=protected area eg way/174172877

If you don't use the preset, and just manually enter the tags one by one then it won't auto-add the building tag, otherwise you'll just need to manually remove the building tag if using the preset in iD.

way/839191190 looks like a building not a tower, so I've changed it and then mapped the satellite dishes as nodes, did you want to take a look and see if that's okay?

way/677568458/history you added layer=-1 but the building is marker as a roof and it's the highest object, I don't see why layer is needed at all here unless you have footways on top of the roof or multiple roofs.

I've reverted the change to Bradleys Head Lighthouse, the other changes seem okay.

No problem making changes, I know accidents like this can happen, so take this as a pointer for next time to watch out for it, and certainly not a discouragement to map. Mapping things from your local knowledge or from compatible sources is fine, if you're unsure about something feel free to ask on one of the community sites (click Australia at https://openstreetmap.community/ to see all the places OSM community is at)

https://osmlab.github.io/osm-deep-history/#/node/1903566739 there are other tags indicating this is a cardinal beacon, by changing it to light_major it looses information that it's cardinal. Are you sure about this change?

the lighthouse you added already existed so I've removed the duplicate you added in changeset/89784100

The iD editor preset automatically adds building=yes, so where you're marking the grounds you need to make sure there is no building=yes tag, you can then add the individual buildings inside the grounds separately. I've fixed this up for the ones in this changeset in changeset/89783972

looks like you accidentally dragged the bus stop into a building https://osmlab.github.io/osm-deep-history/#/node/2626808800 safe to assume that was an accident? I'll move it back.

what's your source for https://osmlab.github.io/osm-deep-history/#/node/247040994 being asphalt, is it from a survey? from the imagery it looks like it could be grass

restored Petersham Reservoir

Move over Pokemon Go edit's this looks like the start of Microsoft Flight Simulator edits...

I've restored the location of YRNS, it's on the hospital building, not the street.

looks like an accidental drag of https://osmlab.github.io/osm-deep-history/#/node/1042094263 so I reverted the change to that node

you could add the building tag, then it will show as a building. an area which is just covered is usually building=roof. Normally I'd tag the whole petrol station grounds as amenity=fuel with all the tags, then a separate building way(s) inside it.