-
Notifications
You must be signed in to change notification settings - Fork 8
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
Have a look at the static warnings and errors currently reported #15
Comments
The current checker has |
Right. I'll take a look, I knew about some of the warnings. For the errors, this is tricky, since these are references to values defined in |
proposal. we could use a JSON, XML, YAML, or MANIFEST file to configure php-analysis, in a similar way we use
And then we'd pass this configuration information down to all the analysis functions. We could have a default template file and users could use their own. Using the defaults of constructor keyword parameters, also sensible defaults can be programmed into the system. That way we always have a Config module and the code would be ready for compilation. |
I would be in favor of moving the config to a yaml file. The problem I ran into before was the following -- I need a way to load the file, but there doesn't seem to be a way for the project to figure out where it is. So, to load the file, I need a path, but to get the path, I would need to put it into a config file. If there is a way around this, that would be great -- I brought it up a while back (when I was starting to update this project) but couldn't really think of a good way around it. |
This is not immediately obvious:
The text was updated successfully, but these errors were encountered: