diff --git a/README.md b/README.md index 9014d70..64633e3 100644 --- a/README.md +++ b/README.md @@ -21,6 +21,7 @@ You will be asked at first-run whether you consent to telemetry being sent to th * Deprecated package names and versions * Chrome user-agent (version of Chrome, OS, CPU) * The number of non-core Atom packages that have been activated +* GitHub username, if a user has authenticated with the GitHub package This information is sent via [Google Analytics][GA] which allows the Atom team to analyze usage patterns and errors in order to help improve Atom. diff --git a/lib/metrics.js b/lib/metrics.js index 7cfcc2e..b5405ab 100644 --- a/lib/metrics.js +++ b/lib/metrics.js @@ -36,7 +36,8 @@ module.exports = { sendEvent: Reporter.sendEvent.bind(Reporter), sendTiming: Reporter.sendTiming.bind(Reporter), addTiming: Reporter.addTiming.bind(Reporter), - sendException: Reporter.sendException.bind(Reporter) + sendException: Reporter.sendException.bind(Reporter), + setGitHubUser: Reporter.setGitHubUser.bind(Reporter) } }, diff --git a/lib/reporter.js b/lib/reporter.js index 9807ac5..6b004bb 100644 --- a/lib/reporter.js +++ b/lib/reporter.js @@ -126,6 +126,10 @@ class Reporter { this.send(params) } + static setGitHubUser (gitHubUser) { + this.getStore().setGitHubUser(gitHubUser) + } + // Private static getStore () { if (!store) { diff --git a/package.json b/package.json index d3fa30c..16d2596 100644 --- a/package.json +++ b/package.json @@ -11,7 +11,7 @@ "providedServices": { "metrics-reporter": { "versions": { - "1.1.0": "provideReporter" + "1.2.0": "provideReporter" } } }, @@ -19,7 +19,7 @@ "fs-plus": "^3.0.0", "grim": "^2.0.1", "node-uuid": "~1.4.7", - "telemetry-github": "0.0.11" + "telemetry-github": "0.0.12" }, "devDependencies": { "standard": "*", diff --git a/spec/metrics-spec.js b/spec/metrics-spec.js index cd9db95..49be4a1 100644 --- a/spec/metrics-spec.js +++ b/spec/metrics-spec.js @@ -703,6 +703,15 @@ describe('Metrics', () => { }) ) + describe('::setGitHubUser', () => + it('sets the gitHubUser', () => { + spyOn(store, 'setGitHubUser') + const gitHubUser = 'beyonce' + reporterService.setGitHubUser(gitHubUser) + expect(store.setGitHubUser).toHaveBeenCalledWith(gitHubUser) + }) + ) + describe('::sendException', () => it('makes a request', () => { reporterService.sendException('desc')