Skip to content
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

Solution to deal with the scope File System required by some smells #60

Open
olegoaer opened this issue Apr 6, 2023 · 0 comments
Open
Assignees
Labels
android Something tied to the android OS 🚀 enhancement New feature or request

Comments

@olegoaer
Copy link
Member

olegoaer commented Apr 6, 2023

File System Scanner

Some smells detection are based on the File System, not on code scanning.

Proposed Solution
Create a new sensor dedicated to that concern, FileSystemSensor, registered at the plugin level. To maintain performance, this sensor should only be triggered once. In the execute() callback, there is a parameter of type SensorContext which allows access to the file system (ex: sc.baseDir())

@olegoaer olegoaer added the android Something tied to the android OS label Apr 6, 2023
@olegoaer olegoaer changed the title How to deal with the scope **File System** requires by some smells How to deal with the scope File System requires by some smells Apr 6, 2023
@olegoaer olegoaer added the 🚀 enhancement New feature or request label Apr 6, 2023
@olegoaer olegoaer changed the title How to deal with the scope File System requires by some smells How to deal with the scope File System required by some smells Apr 6, 2023
@olegoaer olegoaer changed the title How to deal with the scope File System required by some smells Solution to deal with the scope File System required by some smells Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android Something tied to the android OS 🚀 enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants