autogen: fix incorrect dry_run handling in gen_bib_file()

willieyz · willieyz · commit d5f5d676b848 · 2025-10-30T16:46:01.000+08:00
- This commit ported from mldsa-native PR #568 - This commit fixes an issue where the dry_run argument was not working in the gen_bib_file() function. As a result, the CI would not raise an error when BIBLIOGRAPHY.md was missing updates. - The problem was that the dry_run parameter was always set to False, so the dry-run mode was never enabled during execution. Signed-off-by: willieyz <willie.zhao@chelpis.com>
diff --git a/scripts/autogen b/scripts/autogen
@@ -2830,7 +2830,7 @@ def gen_citations(dry_run=False):
                 "Add a citation or remove from BIBLIOGRAPHY.yml."
             )
 
-    gen_bib_file(bibliography, dry_run=False)
+    gen_bib_file(bibliography, dry_run=dry_run)
 
 
 def extract_bytecode_from_output(output_text):

Original file line number	Diff line number	Diff line change
`@@ -2830,7 +2830,7 @@ def gen_citations(dry_run=False):`
`2830`	`2830`	`"Add a citation or remove from BIBLIOGRAPHY.yml."`
`2831`	`2831`	`)`
`2832`	`2832`
`2833`		`- gen_bib_file(bibliography, dry_run=False)`
	`2833`	`+ gen_bib_file(bibliography, dry_run=dry_run)`
`2834`	`2834`
`2835`	`2835`
`2836`	`2836`	`def extract_bytecode_from_output(output_text):`