Two issues with this edit:

1) "Simple changes" isn't a very descriptive comment. If I'm reading the changes correctly, a more accurate comment would be something like "added points of interest".

2) Elevations in OSM are measured in meters, not feet. Your change to the summit of "Colonel Bob" makes it one of the tallest mountains in the state.

Is there a reason why you don't want 13th/Rosamond/West/Westcliff/Deska classified as tertiary? The route has an elevated speed limit, right-of-way over intersecting minor roads, and is classified by the city as an arterial -- all indications that it should be tagged as "tertiary".

Where are you getting the name "Grand Junction" and the presence of a stop sign on a railroad from?

Is this a fire station? Fire stations have their own building type, so they can be indicated as such with an appropriate icon or other marking.

The "name" field should only be used for names, not descriptions. Most parking lots don't have names.

Buildings should not be placed inside other buildings -- it makes it hard for map renderers to draw things correctly. The preferred way to map a building containing multiple businesses is to draw the outline as a generic building, then add a "point" feature for each business.

Is the name of this actually "Residential Development"? The "name" field for objects should only be used for actual names, not descriptions.

Wouldn't it make more sense to draw a "landuse=residential" around the entire apartment complex and name it "Eagle Point Apartments", rather than putting the apartment-complex name on each building individually?

"I Street" is already on the map as a residential road. What were you trying to do here?

The "apartments" tag is for apartment *buildings*, not apartment *complexes*. Applying it to an apartment complex causes the complex to be drawn as a single very large building.

Is this actually a road named "private driveway"? If it's an ordinary driveway, the correct way to mark it as such is a road type of "driveway" with an allowed access of "private". The "name" field should only be used for actual names.

Since when has Pinecroft Natural Area Preserve been a military area?

The classification of places is based on their size, not on the official name. That said, I agree that Pullman is probably large enough to be classified as a city rather than a town.

Landuse tags should indicate "what it's actually used for", not "what it's zoned for".

Based on the Liberty Park map at https://my.spokanecity.org/parks/finder/, it's not part of the park.

I haven't been to the park recently, but aerial imagery shows the block north of Liberty Park Place as a brown, weed-filled lot, where the area south of the road is well-watered, well-maintained grass.

That map dates from 1956, back when Havermale Island was an industrial area surrounding a railroad station. The island was completely re-shaped for the 1974 World's Fair, and is currently undergoing another major renovation.

I've used the "destroyed" lifecycle prefix (osm.wiki/Lifecycle_prefix) for buildings that have been burned down or otherwise destroyed by disaster. It's common for them to be rebuilt on the same foundation with the same roof outline, and this preserves the information for future use.

I don't think it's accurate to describe this section of MLK Way as "open for traffic": as of half an hour ago, the "road closed" construction barriers were still up at the intersection with Sherman.

"boundary=national_park" is a general-purpose "large protected area" tag. The "protected_area" tagging system gives more precision, but until more map renderers understand it, double-tagging national forests as national parks is fairly standard practice.

"boundary=forest" is rarely used, and does not appear to be understood by any map renderer.

The correct tagging for a geyser is "natural=geyser", not "natural=volcano".