@jennyd says: 
> commented-out line 10 in features/step_definitions/mappings_fixture_steps.rb was left in

It's a comment in a standard form inserted by me to indicate what's in `tagged_paths.hashes` when writing a fixture that uses a Cucumber table so that I don't forget what the headers are. I agree it looks probably more like code than it ought to.

- should the "Overwrite existing mappings" option be "Update existing mappings"?
    - it currently sounds misleading when adding to existing tags, because it doesn't remove all the tags which aren't being added to make the tag list consistent with what is entered. I agree that that's the most useful behaviour, but the option name sounds as if it will do the other thing.

- should line 76 in features/step_definitions/mappings_interaction_steps.rb be there?

- lines 294 and 321 in bulk adder spec: it 'has one symbol per failure'
    - these aren't failures, they are the expected successful outcomes

- it's a bit odd still being able to edit tags on the new multiple confirmation page but not the new URL for redirects

Apparently there is snow on the hills where we're staying :) see you all next week!