Hello, I am glad you are editing Open Street Map! One thing you should look into is "descriptive names" on the OSM wiki. For example, the underpass you edited should not have "Underpass" in the name section. If it were named something like "Alice's Memorial Underpass" on a sign nearby or something, that would be appropriate for the name= tag, but only if people actually call it something in that way. The tunnel=yes tag is sufficient to show that it is an underpass.

Why'd you change this to a primary road?

Great job making a changeset!

Hi, I'm glad you're making edits! Remember not to make descriptive names. An example of a descriptive name in this changeset is "truck parking," where the correct tag would be hgv=yes on an area surrounding the parking spaces.

Nice, thanks!

These road classification changes make sense, but I'm wondering what criteria you're using to make the changes?

We generally don't map active construction sites because most data consumers don't update very frequently, so an area could be stuck as construction for a long time. Instead, I wait a couple months until it's finished to map

Why did you include emojis? Is it part of the official name?

Is that how you're supposed to do this? I think the turn restrictions are on the intersection and not as a relation, but I'm not sure.

please write more descriptive changesets.

Nice fast edit! However, driveways are only for private home drives and stuff. Parking lots use service roads and parking aisles.

Hi, it's nice to see you mapping these buildings, but the way you tagged them is incorrect. This wiki page will have more information for how to tag hangars correctly.
building=hangar

Please try to separate large changesets so they take up less space.

*the area should come up to the edge of the road or path

When you're making an area, be sure to not connect it to ways like roads or paths. The way should be in the middle of the road or path, and the area should come up to the road or path.

If the path is deleted, it is likely to be mapped again with less information than before. If no one is allowed on the path, it can be marked with the appropriate access tags. Do you have information on the gun range? We should also map that so people know to stay away.

You removed a road just because it's dangerous? I'd recommend undoing that change and just editing the access tags. Just because a road is dangerous doesn't mean we shouldn't map it.

Ok!

No, the grass was a multipolygon made of two ways.

Is there a reason you made the grass out of two line segments and then used a multipolygon to connect them? Using the same line in two connected multipolygons usually makes it harder to edit in the future.