-
Notifications
You must be signed in to change notification settings - Fork 34
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
New telemetry functionality #1992
Draft
Camwyn
wants to merge
14
commits into
master
Choose a base branch
from
feature/new-telemetry
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
55b513b
Merge branch 'master' into feature/new-telemetry
Camwyn b188e44
initial commit
Camwyn 00876f7
remove logging and add docblock
Camwyn 626fcbc
interim commit
Camwyn 9606ce4
remove logging, alter logic
Camwyn 9fe916a
Break licensing into it's own function.
Camwyn 2c26e50
Restructure the license fetch to just fetch all and loop through them.
Camwyn 4495351
Merge branch 'master' into feature/new-telemetry
Camwyn b1de491
version shenanigans
Camwyn 368c720
Merge branch 'master' into feature/new-telemetry
Camwyn d176c10
PHP 7.4 compat
Camwyn cc30ec7
Update add_license to use $telemetry by reference and just return a b…
Camwyn b2c8d47
Merge branch 'master' into feature/new-telemetry
Camwyn 887c138
Merge branch 'master' into feature/new-telemetry
Camwyn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,13 +1,13 @@ | ||
<?php | ||
/** | ||
* Plugin Name: Tribe Common | ||
* Description: An event settings framework for managing shared options | ||
* Version: 5.2.4 | ||
* Author: The Events Calendar | ||
* Author URI: http://evnt.is/1x | ||
* Text Domain: tribe-common | ||
* License: GPLv2 or later | ||
*/ | ||
/* | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [phpcs] reported by reviewdog 🐶 |
||
Plugin Name: Tribe Common | ||
Description: An event settings framework for managing shared options | ||
Version: 6.6.6 | ||
Author: The Events Calendar | ||
Author URI: http://evnt.is/1x | ||
Text Domain: tribe-common | ||
License: GPLv2 or later | ||
*/ | ||
|
||
/* | ||
Copyright 2009-2015 by The Events Calendar and the contributors | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
[phpcs] reported by reviewdog 🐶
Squiz.Commenting.FunctionComment.Missing
Missing doc comment for function filter_data_args()