Good evening Nathan,
Just a couple of minor nitpicks. I noticed you were removing some implied tags so I thought I would mention that crossing:markings=no is implied by crossing=unmarked and direction=forward is not necessary for highway=giveway on oneway roads since direction=backward is impossible on oneway roads. 
Kind regards,
Kits