Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Remove code generating comments in configuration file #12941

Merged
merged 54 commits into from
Jun 14, 2022
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
54 commits
Select commit Hold shift + click to select a range
b788ea8
add info about config documentation to file header
H-Shay May 18, 2022
370a36b
remove comments from api.py
H-Shay May 18, 2022
74e171b
remove comments from appservice.py
H-Shay May 18, 2022
9fce9af
remove comments from auth.py
H-Shay May 18, 2022
c8ebd3e
remove comments from background_updates.py
H-Shay May 18, 2022
e17b3e5
remove comments from cache.py
H-Shay May 18, 2022
bf7c543
remove comments from captcha.py
H-Shay May 18, 2022
6d30891
remove comments from cas.py
H-Shay May 18, 2022
ad320d3
remove comments from consent.py
H-Shay May 18, 2022
d67d26b
remove comments from database.py
H-Shay May 18, 2022
91bd93d
remove comments from emailconfig.py
H-Shay May 18, 2022
7957222
remove comments from federation.py
H-Shay May 18, 2022
f6e29de
remove comments from federation.py
H-Shay May 18, 2022
004e2ae
remove comments from groups.py
H-Shay May 18, 2022
650332c
remove comments from jwt.py
H-Shay May 18, 2022
b246258
remove comments from modules.py
H-Shay May 19, 2022
11d9f22
remove comments from oembed.py
H-Shay May 19, 2022
9e47700
remove comments from oidc.py
H-Shay May 19, 2022
ee3c658
remove comments from push.py
H-Shay May 19, 2022
6b1cfc3
remove comments from ratelimiting.py
H-Shay May 19, 2022
6babef8
remove comments from redis.py
H-Shay May 19, 2022
22cd3d8
remove comments from retention.py
H-Shay May 19, 2022
29a9e2f
remove comments from room.py
H-Shay May 19, 2022
b449597
remove comments in room_directory.py
H-Shay May 19, 2022
9647714
remove comments from saml2.py
H-Shay May 19, 2022
e090c1e
remove comments from server_notices.py
H-Shay May 19, 2022
b8f32fa
remvoe comments from sso.py
H-Shay May 19, 2022
b6393d4
remove comments from stats.py
H-Shay May 19, 2022
bbce92b
remove comments from tracer.py
H-Shay May 19, 2022
099f79f
remove comments from user_directory.py
H-Shay May 19, 2022
01b88e0
remove comments from voip.py
H-Shay May 19, 2022
b2d0418
remove comments from workers.py
H-Shay May 19, 2022
bb88cc1
remove comments from repository.py
H-Shay May 19, 2022
e797906
remove comments from registration.py
H-Shay May 19, 2022
a925e0e
remove comments from tls.py
H-Shay May 19, 2022
bb52e28
remove comments from server.py
H-Shay May 19, 2022
b97a3e7
remove comments from metrics.py
H-Shay May 19, 2022
e01e1d4
update sample_config_header
H-Shay May 19, 2022
65ff8a0
slight refactor to generate whole config from parts
H-Shay May 31, 2022
60bbdbd
sample config
H-Shay May 31, 2022
2c1211f
remove comments from logger.py
H-Shay May 31, 2022
c66db44
remove comments from key.py
H-Shay May 31, 2022
0925fd3
merge in develop
H-Shay May 31, 2022
573b399
changelog
H-Shay Jun 1, 2022
210e8b1
lint + proper changelog number
H-Shay Jun 1, 2022
408c45e
regenerate sample config
H-Shay Jun 1, 2022
10d2613
remove printed shared secret from tests where unnecessary
H-Shay Jun 1, 2022
04881af
fix issue with registration shared secret not being string
H-Shay Jun 1, 2022
f5b16c1
requested changes
H-Shay Jun 7, 2022
18d7a87
regenerate config
H-Shay Jun 7, 2022
40cbc96
requested changes
H-Shay Jun 9, 2022
bb1b2bb
actually requested change
H-Shay Jun 9, 2022
ffb8f2b
regenerate sample config
H-Shay Jun 9, 2022
2348e47
Merge branch 'develop' into shay/bye_bye_comments
H-Shay Jun 9, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion tests/app/test_homeserver_start.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
class HomeserverAppStartTestCase(ConfigFileTestCase):
def test_wrong_start_caught(self):
# Generate a config with a worker_app
self.generate_config()
self.generate_config_and_remove_lines_containing("registration_shared_secret")
DMRobertson marked this conversation as resolved.
Show resolved Hide resolved
# Add a blank line as otherwise the next addition ends up on a line with a comment
self.add_lines_to_config([" "])
self.add_lines_to_config(["worker_app: test_worker_app"])
Expand Down
2 changes: 1 addition & 1 deletion tests/config/test_load.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ def test_load_succeeds_if_macaroon_secret_key_missing(self):
)

def test_disable_registration(self):
self.generate_config()
self.generate_config_and_remove_lines_containing("registration_shared_secret")
self.add_lines_to_config(
["enable_registration: true", "disable_registration: true"]
)
Expand Down