Skip to content

Commit d7916ef

Browse files
committed
Merge 'Add CI check to ensure no presets are password-locked' (OoTRandomizer#2386)
2 parents 25a55d7 + 2fa95ac commit d7916ef

2 files changed

Lines changed: 7 additions & 4 deletions

File tree

CI.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -114,12 +114,15 @@ def check_hell_mode_tricks(fix_errors: bool = False) -> None:
114114
print(file=file)
115115

116116

117-
def check_preset_spoilers(fix_errors: bool = False) -> None:
118-
# Check to make sure spoiler logs are enabled for all presets.
117+
def check_preset_settings(fix_errors: bool = False) -> None:
118+
# Check to make sure password lock is disabled and spoiler logs are enabled for all presets.
119119
with open(data_path('presets_default.json'), encoding='utf-8') as f:
120120
presets = json.load(f)
121121

122122
for preset_name, preset in presets.items():
123+
if preset['password_lock']:
124+
error(f'{preset_name} preset is password-locked', True)
125+
preset['password_lock'] = False
123126
if not preset['create_spoiler']:
124127
error(f'{preset_name} preset does not create spoiler logs', True)
125128
preset['create_spoiler'] = True
@@ -259,7 +262,7 @@ def run_ci_checks() -> NoReturn:
259262
check_code_style(args.fix)
260263
check_presets_formatting(args.fix)
261264
check_table_sizes()
262-
check_preset_spoilers(args.fix)
265+
check_preset_settings(args.fix)
263266
check_message_duplicates()
264267

265268
exit_ci(args.fix)

version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
__version__ = '8.3.48'
1+
__version__ = '8.3.49'
22

33
# This is a supplemental version number for branches based off of main dev.
44
supplementary_version = 0

0 commit comments

Comments
 (0)