Skip to content

Commit c0f4531

Browse files
authored
Merge pull request zeusops#7 from zeusops/pretty
Change config upload to pretty-print JSON
2 parents c881c90 + 7d523e4 commit c0f4531

File tree

3 files changed

+6
-2
lines changed

3 files changed

+6
-2
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@ The project uses semantic versioning (see [semver](https://semver.org)).
55

66
## [Unreleased]
77

8+
### Fixed
9+
10+
- Uploaded missions are now pretty-printed, to make it reviewable.
11+
812
## v0.4.0 - 2025-03-12
913

1014
### Added

src/zeusops_bot/reforger_config_gen.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def zeus_upload(
5858
modded_config_dict = patch_file(base_config_content, modlist, scenario_id)
5959
target_filepath = as_config_file(self.target_dest, filename)
6060
# Create the file itself
61-
target_filepath.write_text(json.dumps(modded_config_dict))
61+
target_filepath.write_text(json.dumps(modded_config_dict, indent=4))
6262
return target_filepath
6363

6464
def zeus_set_mission(self, filename):

tests/test_list_missions.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,4 +26,4 @@ def test_list_missions(
2626
# When Zeus calls "/zeus-list"
2727
result: list[str] = config_gen.list_missions()
2828
# Then a list of mission names is returned
29-
assert result == mission_names, "Should return mission names"
29+
assert set(result) == set(mission_names), "Should return mission names"

0 commit comments

Comments
 (0)