Skip to content

adds --ignore functionality to ignore files #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

rathi-yash
Copy link
Owner

@rathi-yash rathi-yash commented Mar 5, 2025

Description

This PR introduces the --ignore feature, allowing users to specify paths that should be skipped during dead code detection by deadcode-detective.

  • Added the --ignore flag to exclude user-specified paths from analysis.
  • Implemented error handling for path expansion issues that occur when using Git Bash.
  • This is an interim solution—once a configuration file is introduced, path handling will be managed more effectively.

Future Considerations

  • The config file implementation will provide a more robust way to handle ignored paths, eliminating the need for manual flag input.

Type of Change

  • New feature

Checklist

  • My code follows the project's coding style and guidelines.
  • I have performed a self-review of my code.
  • I have commented my code where necessary.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have updated the documentation as needed.

Additional Information

image
image
image

@rathi-yash rathi-yash added the enhancement New feature or request label Mar 6, 2025
@rathi-yash rathi-yash moved this to In Progress in Deadcode-Detective Board Mar 6, 2025
@rathi-yash rathi-yash marked this pull request as ready for review March 19, 2025 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

1 participant