I don’t think I would subscribe to this here, the military key is about military facilities and a civilian bunker does not fall under it.
Yes, we carry some luggage from the past: there is a list of values or k/v-combinations that are explicitly defined to mean something that is somehow in contrast to the rest, e.g. the key. We don’t help ourselves if we continue loosening the meaning of keys and values by explicitly defining them to include stuff that doesn’t fall under the current definitions or would not be expected to be covered by the tag name alone, it would make it all much more difficult for everybody. Well, I don’t say never, but at least it should not become the default.
As an addition to the document linked and quoted by loffa, here the information for Switzerland (in english but of course you can switch to any of our languages if you understand them better) Shelters for the population
Those shelters built in normal houses are completely paid and maintained by the building owner - the state just says what you have to do…
Please do not go for anything like building=bunker if we go for a change, only the bigger installations in Switzerland would be correctly tagged that way, most of them are just a room (or a few rooms) in the basement of a building that should be tagged with another building= tag
I just checked the comments in that changeset.
It seems Ok. The mapper who added it seems not to be a Newbie at all and the one who changed them to fix better seems from the local community.