Skip to content

Remove unused imports, fix no-op copies, and fix typo in scripts#7038

Merged
szarnyasg merged 1 commit into
duckdb:mainfrom
arijitroy003:cleanup/unused-imports-and-typos
Jul 20, 2026
Merged

Remove unused imports, fix no-op copies, and fix typo in scripts#7038
szarnyasg merged 1 commit into
duckdb:mainfrom
arijitroy003:cleanup/unused-imports-and-typos

Conversation

@arijitroy003

Copy link
Copy Markdown
Contributor

Summary

  • Remove unused imports across several scripts: csv and io from generate_config_docs.py, sys from copy_stable_to_preview.py, os and jinja2.optimizer.Optimizer from generate_profiling_docs.py, glob from redirect.py
  • Remove no-op shutil.copy calls where source and destination are identical (_data/menu_docs_current.json to itself) in copy_stable_to_preview.py and archive_docs.py
  • Fix CUMALATIVE typo to CUMULATIVE in variable name in generate_profiling_docs.py
  • Remove duplicate python-frontmatter entry from requirements.txt

Test plan

  • Verified each issue exists in the current codebase before fixing
  • Confirm no behavioral changes: unused imports are never referenced, no-op copies had identical src/dst, typo fix is internal variable naming only

- Remove unused imports: csv, io (generate_config_docs.py), sys
  (copy_stable_to_preview.py), os, jinja2.optimizer.Optimizer
  (generate_profiling_docs.py), glob (redirect.py)
- Remove no-op shutil.copy where src == dst in copy_stable_to_preview.py
  and archive_docs.py
- Fix CUMALATIVE typo to CUMULATIVE in generate_profiling_docs.py
- Remove duplicate python-frontmatter entry from requirements.txt
@szarnyasg

Copy link
Copy Markdown
Collaborator

Thanks!

@szarnyasg
szarnyasg merged commit 0051dc1 into duckdb:main Jul 20, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants