Skipping is currently tied to reindexing

[cdown]

I've been thinking about adding a --fast mode which would skip reindexing, skip checks, and use a simpler regex (and possibly more things). However, right now I've noticed that these are coupled since we only do _should_skip_sub inside sort_and_reindex.

We should separate these out.