carbon-language/carbon-lang
1
0
Fork 0
mirror of https://github.com/carbon-language/carbon-lang.git synced 2026-10-05 16:31:04 +01:00
Code Issues Packages Projects Releases 10 Wiki Activity

Correctly remove ## TODO section from proposals (#2207)

Browse Source 
Fix bug introduced in #2139
This commit is contained in:
josh11b
2022-09-22 08:20:07 -07:00
committed by GitHub
parent 1dcd15380c
commit e941b90ac9
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
proposals/scripts/new_proposal.py
+1 -1
View File
@@ -98,7 +98,7 @@ def _fill_template(template_path: str, title: str, pr_num: int) -> str:
r"\g<1>%d" % pr_num,
content,
)
content = re.sub(r"\n## TODO(?:|\n)*?(\n## )", r"\1", content)
content = re.sub(r"\n## TODO(?:.|\n)*?(\n## )", r"\1", content)
return content
proposals/scripts/new_proposal_test.py
+3 -1
View File
@@ -50,7 +50,9 @@ class TestNewProposal(unittest.TestCase):
"pull/123)" in content,
content,
)
self.assertTrue("\n## Abstract\n\n" in content, content)
self.assertTrue(
"<!-- tocstop -->\n\n## Abstract\n\n" in content, content
)
def test_run_success(self):
new_proposal._run(["true"])