search

PLUS-POSTECH / soma

Cross-platform CTF problem container manager
Apache License 2.0
24 stars 3 forks source link

Refactor module structures #92

Closed KSAlpha closed 5 years ago

KSAlpha commented 5 years ago

RepositoryManager's module name (repo::repo_manager::RepositoryManager) is way to verbose. We can make it shorter by removing redundant repo_ prefix (i.e. repo::manager::RepositoryManager).

Also consider moving some structs like Manifest and SolidManifest in soma::repo to soma::repo::problem or soma::problem.

Qwaz commented 5 years ago

Note that repo re-exports RepositoryManager, so it is possible to write soma::repo::RepositoryManager.

Qwaz commented 5 years ago

I'm thinking about something like this.

soma
├ repo
│ ├ manager
│ ├ repository
│ └ problem
└ manifest
   ├ manifest
   └ binary_config