Hi and welcome to OSM!
Important note - we are not allowed to use municipal gis data for OSM mapping. Without proper approval this is a serious copyright violation. You can only use approved sources (usually available in the editors) like bing and esri). If you continue to use unapproved sources - all your work might be reverted and your user will be blocked.

Hi, you added here roundabout (way/1387430145) which I can't find on any available satellite imagery. Can you, please, confirm your sources and that this was not mistake?

hi, also the 24/6 is not standard format for hours (in node/12816126410) I'm afraid you'll need to do something like "Mo-Su: 00:00-24:00; Sa: off"

Hi, you also added strange round building way/1384151277 I'd like to know what kind of building is this? I don't see anything similar on street view maps.

I understand, but Tashoot is definitely originated from Hebrew...
Never mind, if they like it this way - let it be.
thanks!

I'd also add amenity=boat_rental, guess this will not conflict with the club tag

heeey, should not the name tag be in Hebrew? ;)
Herzliya Pituach is a special case, but still it's in Israel :)

Hi, you changed street name
הר הארבל
to
ארבל

can you, please, confirm which one is correct and what is your source? This change happens second time, so I'd like some proofs from the ground if you have

I personally don't see any use for these "note=street location" notes. Do you? Let's not add them?

Hi Omer, if there is an only single building nearby, I'd add address to the building instead of creation of an address node

Hi, I just corrected an old mistake, removed name tag from roundabouts junctions. By a convention name tag should be used only in case roundabout has a dedicated name.

See more here:
junction=roundabout#The_roundabout_itself

Thanks for response!
Yes, this exact definition in wiki made me think that city must be firstly divided to suburbs and only then each suburb can be sub-divided to neighbourhoods.

Statistically it looks like I'm not the only one which uses suburb for שכונה, so I'd like to try to align this with the community.

At the moment we'll have a good agreement there, I promise to fix the Raanana if needed.

https://community.openstreetmap.org/t/place-is-suburb-quarter-or-neighbourhood/129560

Hi, why place=neighbourhood? let's discuss before you change them all!

ok, just as reminder, you can't use any maps for which you didn't get any explicit official approval. So, MoT maps is a big no-no. Copyrights is a serious violation which might make all you work removed.

don't rush, wait until it open, and then you'll be able to record proper GPX or we'll have updated Bing maps and can map it precisely.

Hello and thank you for contribution!

You added here a separate node for the park while whole park is already exists as a shape way/1197962918

This created unneeded duplicates. Please be sure to always check the area to ensure your new feature is not yet on the map.

Hi, just to be sure, if this rail is still under construction, what kind of sources do you use to refine the way shape?

Welcome to OSM and thank you for contribution!
Just to ensure, is this a simple water tap or a drinking water fountain?
if the latter - then amenity=drinking_water fits here better
(amenity=drinking_water)

Hi, the default "name" tag should be always in a language used by most of the local population. Specifically here it's 1000% Hebrew.

happily this was fixed by other mappers long time ago.