Skip to content

Commit

Permalink
fix: Change telemetry command line message (#1067)
Browse files Browse the repository at this point in the history
* fix: Change telemetry command line message

Signed-off-by: Flavius Lacatusu <[email protected]>

* fix: Minishift tests

Signed-off-by: Flavius Lacatusu <[email protected]>
  • Loading branch information
flacatus authored Jan 20, 2021
1 parent dc032a1 commit f720077
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/e2e-minishift-operator.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
- uses: actions/checkout@v1
- name: Install and start minishift OCP 3.11 cluster
run: |
brew cask install minishift
brew install minishift
export MINISHIFT_GITHUB_API_TOKEN=${{ secrets.GITHUB_TOKEN }}
minishift start --memory=5500 --vm-driver=virtualbox
- name: Generate minishift certificates
Expand Down
3 changes: 1 addition & 2 deletions src/hooks/analytics/analytics.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ import { IConfig } from '@oclif/config'
import { cli } from 'cli-ux'

import { ConfigManager } from '../../api/config-manager'
import { getProjectName } from '../../util'

import { SegmentAdapter, SegmentProperties } from './segment-adapter'

Expand All @@ -29,7 +28,7 @@ export const hook = async (options: { command: string, flags: any, config: IConf

// Prompt question if user allow chectl to collect data anonymous data.
if (!options.flags.telemetry && !segmentTelemetry) {
const confirmed = await cli.confirm(`${getProjectName()} would like to collect data about how users use cli commands. Participation is voluntary and when you choose to participate chectl automatically sends statistic usage about how you use the cli [y/n]:`)
const confirmed = await cli.confirm('Enable CLI usage data to be sent to Red Hat online services. [y/n]')
segmentTelemetry = confirmed ? 'on' : 'off'
configManager.setProperty(SegmentProperties.Telemetry, segmentTelemetry)
}
Expand Down

0 comments on commit f720077

Please sign in to comment.