-
Notifications
You must be signed in to change notification settings - Fork 7
Open
Labels
testingSomething needs to be testedSomething needs to be tested
Description
In PR #19 basic configuration was implemented. However, the configuration system is really decoupled. You need to:
- Define the correct configuration key, with the correct type, in
package.json - Define the correct member in the
Configinterface.
This process is quite error-prone. Complete tests in config.test.ts to:
- Check that every
contributes.configurationkey inpackage.jsonstarts withplastic-scm. - Check that every
contributes.configurationkey exists in theConfiginterface. - Check that every property defined in the
Configinterface exists undercontributes.configuration
If its possible, it would be extremely nice to...
- Check that every configuration key in
package.jsondefines the same type used later on theConfiginterface.
Metadata
Metadata
Assignees
Labels
testingSomething needs to be testedSomething needs to be tested