Remove JobSearchIndex (unused class).

[bdice]

Description

Remove unused JobSearchIndex class.

This class isn't used and is untested. I think it can be removed without any issue. However, we may wish to deprecate this class in signac 1.8 for good measure.

Motivation and Context

signac 2.0 cleanup.

Types of Changes

• Documentation update
• Bug fix
• New feature
• Breaking change¹

¹The change breaks (or has the potential to break) existing functionality.

Checklist:

• I am familiar with the Contributing Guidelines.
• I agree with the terms of the Contributor Agreement.
• My name is on the list of contributors.
• My code follows the code style guideline of this project.
• The changes introduced by this pull request are covered by existing or newly introduced tests.

If necessary:

• I have updated the API documentation as part of the package doc-strings.
• I have created a separate pull request to update the framework documentation on signac-docs and linked it here.
• I have updated the changelog and added all related issue and pull request numbers for future reference (if applicable). See example below.

[codecov]

Codecov Report

Merging #587 (1d36212) into next (c6e059e) will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             next     #587      +/-   ##
==========================================
+ Coverage   82.41%   82.62%   +0.21%     
==========================================
  Files          54       54              
  Lines        5845     5824      -21     
  Branches     1097     1090       -7     
==========================================
- Hits         4817     4812       -5     
+ Misses        785      769      -16     
  Partials      243      243              

Impacted Files	Coverage Δ
signac/contrib/project.py	87.66% <ø> (+1.74%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c6e059e...1d36212. Read the comment docs.

[vyasr]

For consistency, can you make a PR to deprecate this on master before we merge this PR? I don't want to lose track of that.

[bdice]

For consistency, can you make a PR to deprecate this on master before we merge this PR? I don't want to lose track of that.

I’m tracking those with the label requires-1.x-deprecation and am planning to do follow-up work that handles those deprecations for a number of these removals at once.

[bdice]

@vyasr I created #600 for the corresponding deprecation of this class.