ToolboxHub / ToolboxRegistry

Public Toolbox Configurations for use with ToolboxToolbox
The Unlicense
3 stars 17 forks source link

Favor "include" records for toolboxes know to the ToolboxRegistry #16

Open benjamin-heasly opened 7 years ago

benjamin-heasly commented 7 years ago

We have some configurations in the ToolboxRegistry that contain duplicate records, for toolboxes that are already known to the registry.

For example, the isetbio configuration contains a record for the UnitTestToolbox, which points at the UnitTestToolbox git repository.

But UnitTestToolbox is already known to the registry, with its own configuration that points to the same place.

Instead of duplicating this information, the isetbio configuration should refer to UnitTestToolbox with an "include" record that can be resolved to the already-known UnitTestToolbox configuration. For example:

{
    "name": "UnitTestToolbox",
    "type": "include"
}

Some reasons why including already-known configurations would be good:

We should review the registry and update configurations so that toolboxes known to the registry are referred to as "include" records.

DavidBrainard commented 7 years ago

I did some of this, but need to make sure it is all done before closing.