Separate directories in 'lib/datasets'

[heronshoes]

The current lib/datasets directory is a mix of .rb files for the common engine and .rb files for the individual datasets.

I think separating these would improve the outlook.

Proposal 1:

lib/
  datasets/ # includes common/ and each .rb for datasets
    common/

Proposal 2:

lib/
  datasets/ # includes each .rb for datasets
  common/   # includes common engines

[kou]

Proposal 2 isn't acceptable. We should put all files under datasets/ to avoid conflict.

I don't like common. If we use datasets/common/, Datasets::Error should be renamed to Datasets::Common::Error, right? Datasets::Common::Error is strange for me. I want to use Datasets::Error as an user.

If someone have a proposal, please share it.

[heronshoes]

Ah, I didn't think about errors.

Is this work?

lib/
  datasets/ # includes common engines
    (some_good_name)/ # includes each .rb for datasets