diff --git a/.scrutinizer.yml b/.scrutinizer.yml index 016129c7629..a176c114cb8 100644 --- a/.scrutinizer.yml +++ b/.scrutinizer.yml @@ -13,36 +13,7 @@ filter: paths: { } tools: - # Similar code detection - php_sim: - enabled: true - min_mass: 16 - filter: - excluded_paths: - - 'build/*' - - 'dev/*' - - 'doc/*' - - 'test/*' - - 'htdocs/includes/*' - paths: { } - - # php_depend - php_pdepend: - enabled: true - configuration_file: null - suffixes: - - php - excluded_dirs: { } - filter: - excluded_paths: - - 'build/*' - - 'dev/*' - - 'doc/*' - - 'test/*' - - 'htdocs/includes/*' - paths: { } - - # php_analyzer + # php_analyzer. Doc on https://scrutinizer-ci.com/docs/tools/php/php-analyzer/ php_analyzer: enabled: true extensions: @@ -56,11 +27,131 @@ tools: - 'test/*' - 'htdocs/includes/*' paths: { } - path_configs: { } + config: + parameter_reference_check: + enabled: true + checkstyle: + enabled: false + no_trailing_whitespace: true + naming: + enabled: true + local_variable: ^[a-z][a-zA-Z0-9]*$ + abstract_class_name: ^Abstract|Factory$ + utility_class_name: Utils?$ + constant_name: ^[A-Z][A-Z0-9]*(?:_[A-Z0-9]+)*$ + property_name: ^[a-z][a-zA-Z0-9]*$ + method_name: ^(?:[a-z]|__)[a-zA-Z0-9]*$ + parameter_name: ^[a-z][a-zA-Z0-9]*$ + interface_name: ^[A-Z][a-zA-Z0-9]*Interface$ + type_name: ^[A-Z][a-zA-Z0-9]*$ + exception_name: ^[A-Z][a-zA-Z0-9]*Exception$ + isser_method_name: ^(?:is|has|should|may|supports) + unreachable_code: + enabled: true + check_access_control: + enabled: true + typo_checks: + enabled: true + check_variables: + enabled: true + check_calls: + enabled: true + too_many_arguments: true + missing_argument: true + argument_type_checks: lenient # Allowed Values: "disabled", "lenient", "strict" + suspicious_code: + enabled: true + overriding_parameter: false + overriding_closure_use: true + parameter_closure_use_conflict: true + parameter_multiple_times: true + non_existent_class_in_instanceof_check: true + non_existent_class_in_catch_clause: true + assignment_of_null_return: true + non_commented_switch_fallthrough: true + non_commented_empty_catch_block: true + overriding_private_members: true + use_statement_alias_conflict: true + precedence_in_condition_assignment: true + dead_assignments: + enabled: true + verify_php_doc_comments: + enabled: false + parameters: true + return: true + suggest_more_specific_types: true + ask_for_return_if_not_inferrable: true + ask_for_param_type_annotation: true + loops_must_use_braces: + enabled: true + check_usage_context: + enabled: true + simplify_boolean_return: + enabled: false + phpunit_checks: + enabled: false + reflection_checks: + enabled: true + + # Checks Common Precedence Mistakes + precedence_checks: + enabled: true + assignment_in_condition: true + comparison_of_bit_result: true + basic_semantic_checks: + enabled: true + unused_code: + enabled: true + deprecation_checks: + enabled: true + useless_function_calls: + enabled: true + metrics_lack_of_cohesion_methods: + enabled: true + metrics_coupling: + enabled: true + stable_code: + namespace_prefixes: [] + classes: [] + doctrine_parameter_binding: + enabled: false + doctrine_entity_manager_injection: + enabled: false + symfony_request_injection: + enabled: false + doc_comment_fixes: + enabled: true + reflection_fixes: + enabled: false + use_statement_fixes: + enabled: true + remove_unused: true + # Whether you would like multiple imports in one USE statement to be preserved, e.g. ``use A, B;``. + preserve_multiple: false + # Whether you would like to preserve blank lines between use statements. + preserve_blanklines: false + order_alphabetically: false + # To use specific config for a specific path, use path_configs: (see example on page https://scrutinizer-ci.com/docs/configuration/tool_config_structure) + + # php_depend + php_pdepend: + enabled: false + configuration_file: null + suffixes: + - php + excluded_dirs: { } + filter: + excluded_paths: + - 'build/*' + - 'dev/*' + - 'doc/*' + - 'test/*' + - 'htdocs/includes/*' + paths: { } # change tracking php_changetracking: - enabled: true + enabled: false bug_patterns: - '\bfix(?:es|ed)?\b' feature_patterns: @@ -75,8 +166,21 @@ tools: - 'htdocs/includes/*' paths: { } + # Similar code detection + php_sim: + enabled: false + min_mass: 30 + filter: + excluded_paths: + - 'build/*' + - 'dev/*' + - 'doc/*' + - 'test/*' + - 'htdocs/includes/*' + paths: { } + # Coding-Style / Bug Detection - js_hint: true + js_hint: false before_commands: { }