Skip to content

improve: fix eslint config sourceType, add devDependencies, add node_modules to gitignore#40

Open
Coding-Dev-Tools wants to merge 1 commit into
mainfrom
improve/configdrift-1782893596
Open

improve: fix eslint config sourceType, add devDependencies, add node_modules to gitignore#40
Coding-Dev-Tools wants to merge 1 commit into
mainfrom
improve/configdrift-1782893596

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

Automated improvement by dev-engineer

  • Fixed eslint.config.mjs sourceType from 'commonjs' to 'module' (file uses ESM import syntax)
  • Added devDependencies (@eslint/js, eslint, globals) to package.json
  • Removed duplicate devDependencies key from package.json
  • Added node_modules/ to .gitignore

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

Would reformat: src/configdrift/cli.py
Would reformat: src/configdrift/diff.py
Would reformat: src/configdrift/loader.py
Would reformat: tests/test_cli.py
Would reformat: tests/test_coverage_gaps.py
Would reformat: tests/test_diff.py
Would reformat: tests/test_loader.py
7 files would be reformatted, 2 files already formatted

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 3 file(s) changed

 .gitignore        |   1 +
 eslint.config.mjs |   2 +-
 package.json      | 107 +++++++++++++++++++++++++++---------------------------
 3 files changed, 56 insertions(+), 54 deletions(-)

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

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

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