The uses of these classes were deleted in commit acea24d0ef1acdac036e29ba1dd1032bef4755d0 in 2022. Clear them out so they don't distract from on-going work
Removals
Classes {BMI,CSV,NetCDF}DataSelector that were all empty subclasses of CatchmentAggrDataSelector
Changes
{CSV,NetCDF}DataProvider tests modified to use base class instead of specific derived classes
Testing
CI
Checklist
[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows project standards (link if applicable)
[x] Passes all existing automated tests
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future todos are captured in comments
[x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[x] Reviewers requested with the Reviewers tool :arrow_right:
The uses of these classes were deleted in commit acea24d0ef1acdac036e29ba1dd1032bef4755d0 in 2022. Clear them out so they don't distract from on-going work
Removals
{BMI,CSV,NetCDF}DataSelectorthat were all empty subclasses ofCatchmentAggrDataSelectorChanges
{CSV,NetCDF}DataProvidertests modified to use base class instead of specific derived classesTesting
Checklist