Skip to content

Commit bee3079

Browse files
committed
config: move default interactive value to validate_config()
Per @ryneeverett's request, the value should be removed from the schema.py. This breaks most of the tests, adding the `interactive = False` to validate_config() fixes most of them. Some are still broken, for which #1193 was created.
1 parent a61bb2f commit bee3079

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

bugwarrior/config/schema.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ class MainSectionConfig(BaseConfig):
138138

139139
# added during configuration loading
140140
#: Interactive status.
141-
interactive: bool = False
141+
interactive: bool
142142

143143
@computed_field
144144
@property

tests/config/test_validation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ class TestValidation(ConfigTest):
1010
def setUp(self):
1111
super().setUp()
1212
self.config = {
13-
'general': {'targets': ['my_service', 'my_kan', 'my_gitlab']},
13+
'general': {'targets': ['my_service', 'my_kan', 'my_gitlab'], 'interactive': False},
1414
'my_service': {
1515
'service': 'github',
1616
'login': 'ralph',

0 commit comments

Comments
 (0)