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

a simple option for handling external libraries #66

Open
the-liquid-metal opened this issue May 9, 2020 · 3 comments
Open

a simple option for handling external libraries #66

the-liquid-metal opened this issue May 9, 2020 · 3 comments

Comments

@the-liquid-metal
Copy link

Rather than sets $conf->t_ignore_* manually, it is better to provide a single config entry $conf->library_paths which contains all the path of libraries. The steps are:

  • scans those paths
  • push founded class into $conf->t_ignore_classes and so on.

so $conf->t_ignore_* entered by user are minimal.

@KminekMatej
Copy link

WIP at my fork https://github.com/KminekMatej/yakpro-po

@sedimentation-fault
Copy link

This is probably the most important feature request for this project. It is always a PITA having to ripgrep for "function ..." in newly included external libraries and exclude the names of their methods by putting them manually in the corresponding array of the configuration file - same for classes.

@KminekMatej
Copy link

KminekMatej commented Feb 29, 2024

I started to work on this feature, however its nearly impossible to detect external library usage on static analysis level without tools closely experienced for that case, like PHPStan.

I will try to implement this analysis using PHPStan used to detect external libraries, but I am afraid the changes which would have to be done in current project would be so high it would probably be easier to write it as an extension for PHPStan rather that updating yakpro-po.

Still in development though - gonna be a loong ride :)

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

No branches or pull requests

3 participants