Remove unnecessary context block from feature
Rename version.rb to version_number.rb

this avoids clashing as the rails autoloader expects to find the
definition of the class `Peoplefinder::Version` in the file
lib/peoplefinder/version.rb
Remove person from search index after deleting
Database schema should not be committed

The dummy app should be a test of the whole engine, including its
provided database migrations which should properly set up the
application starting from a fresh application without any existing
schema.

The test process should run the database migrations as its first step.
Rubocop Style/StringLiterals: prefer single quotes
Rubocop: Style/SpaceInsideBlockBraces
Some tidying up prompted by rubocop cyclomatic complexity warning
Unravel SQL literal a bit to reduce line length

but increased method length, so increase rubocop method length limit.
Ensure that this directory exists

Rails attempts to write the structure.sql file here when migrations are run
so the directory needs to exist.
Disable Style/SingleLineBlockParams

dont think we want to enforce block params |a, e| for all #inject calls, do we?
Disable Style/RedundantSelf cop
Avoid perl-style backref

I agree these are potentially confusing syntax and easily missed
Simplify SearchIndex#search
Move `ImportOne` class out as `PersonIndexer`

the indexing operation is a little bit complex so deserves a separate
file
Add GIN index to search_index.document

the GIN index speeds up full text searching[1].

[1] http://www.postgresql.org/docs/9.3/static/textsearch-indexes.html