Skip to content

improve: fix author name and add multi-line export list detection#41

Merged
Coding-Dev-Tools merged 1 commit into
masterfrom
improve/deadcode-20260624-160935
Jun 29, 2026
Merged

improve: fix author name and add multi-line export list detection#41
Coding-Dev-Tools merged 1 commit into
masterfrom
improve/deadcode-20260624-160935

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

Automated improvement by dev-engineer. Changes:

  • Fix author name in pyproject.toml (DevForge -> Revenue Holdings)
  • Fix multi-line export list detection in scanner.py (handle comments, aliases, multi-line blocks)
  • Add comprehensive test coverage for multi-line export lists (5 new tests)
  • Update CLI edge case tests to use CliRunner instead of subprocess
  • Add cowork-auto-pr workflow for automated PR creation

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

Would reformat: src/deadcode/__main__.py
Would reformat: src/deadcode/cli.py
Would reformat: src/deadcode/scanner.py
Would reformat: tests/test_cli_edge_cases.py
Would reformat: tests/test_config_and_fixes.py
Would reformat: tests/test_scanner.py
6 files would be reformatted, 3 files already formatted

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 0 file(s) changed


Verdict: ⚠️ Warnings — Lint/format issues found. Recommend fixing before merge.

Automated by Coding-Dev-Tools/.github reusable workflow.

The author name fix and multi-line export detection are already on master.
This commit updates the PR branch to resolve merge conflicts.
@Coding-Dev-Tools Coding-Dev-Tools force-pushed the improve/deadcode-20260624-160935 branch from fdc1c27 to e9a68fb Compare June 29, 2026 09:57
@Coding-Dev-Tools Coding-Dev-Tools merged commit 8d66692 into master Jun 29, 2026
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.

1 participant