-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix Tracking changes #57
Conversation
Changed 'ts_tracker_data' filter name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code analysis identified issues
Tyche Softwares has identified potential problems in this pull request during automated scanning. We recommend reviewing the issues noted and that they are resolved.
phpcs scanning turned up:
🚫 2 errors
Powered by Tyche Softwares
// WooCommerce version. | ||
$data['wc_plugin_version'] = WC()->version; | ||
$data['active_plugins'] = $active_plugins; | ||
$data['inactive_plugins'] = $plugins; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 Error: Spaces must be used for mid-line alignment; tabs are not allowed (Universal.WhiteSpace.DisallowInlineTabs.NonIndentTabsUsed).
// WooCommerce version. | ||
$data['wc_plugin_version'] = WC()->version; | ||
$data['active_plugins'] = $active_plugins; | ||
$data['inactive_plugins'] = $plugins; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚫 Error: Whitespace found at end of line (Squiz.WhiteSpace.SuperfluousWhitespace.EndLine).
Dismissing review as all inline comments are obsolete by now
Scanning latest commit did not yield any new issues. Please have a look at older feedback still existing (commit-ID: be2ddb1) Powered by Tyche Softwares |
Changed 'ts_tracker_data' filter name