Skip to content

Commit 70bfa90

Browse files
authored
refactor: rename telemetry config to anonymous_telemetry (#180)
1 parent 02b1123 commit 70bfa90

4 files changed

Lines changed: 19 additions & 9 deletions

File tree

‎schema/ferrflow.json‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,14 @@
2323
"description": "Main branch name.",
2424
"default": "main"
2525
},
26+
"anonymous_telemetry": {
27+
"type": "boolean",
28+
"description": "Send anonymous usage statistics to improve FerrFlow. No identifying data is collected.",
29+
"default": true
30+
},
2631
"telemetry": {
2732
"type": "boolean",
28-
"description": "Send anonymous usage statistics to improve FerrFlow.",
33+
"description": "Deprecated: use anonymous_telemetry instead.",
2934
"default": true
3035
},
3136
"versioning": {

‎src/config.rs‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,8 @@ pub struct WorkspaceConfig {
6060
pub remote: String,
6161
#[serde(default = "default_branch")]
6262
pub branch: String,
63-
#[serde(default = "default_telemetry")]
64-
pub telemetry: bool,
63+
#[serde(default = "default_telemetry", alias = "telemetry")]
64+
pub anonymous_telemetry: bool,
6565
#[serde(default)]
6666
pub versioning: VersioningStrategy,
6767
#[serde(alias = "tagTemplate")]
@@ -727,7 +727,7 @@ pub fn init(format: Option<ConfigFileFormat>) -> Result<()> {
727727
println!("Created {filename}");
728728
println!("Run: ferrflow check");
729729

730-
if config.workspace.telemetry {
730+
if config.workspace.anonymous_telemetry {
731731
telemetry::send_event(telemetry::EventType::Init, None, None, None, None);
732732
}
733733

@@ -1667,7 +1667,7 @@ format = "toml"
16671667
let json = r#"{"workspace":{"remote":"origin"},"package":[]}"#;
16681668
let config: Config = serde_json::from_str(json).unwrap();
16691669
assert_eq!(config.workspace.remote, "origin");
1670-
assert!(config.workspace.telemetry);
1670+
assert!(config.workspace.anonymous_telemetry);
16711671
assert!(config.workspace.auto_merge_releases);
16721672
}
16731673

‎src/monorepo.rs‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ pub fn check(config_path: Option<&Path>, verbose: bool) -> Result<()> {
2727

2828
let result = run_release_logic(&root, &config, true, verbose);
2929

30-
if config.workspace.telemetry {
30+
if config.workspace.anonymous_telemetry {
3131
telemetry::send_event(telemetry::EventType::Check, None, None, None, None);
3232
}
3333

@@ -274,7 +274,7 @@ fn run_release_logic(root: &Path, config: &Config, dry_run: bool, verbose: bool)
274274
auto_stage_new_files(&repo, &before, &mut files_to_commit);
275275
}
276276

277-
if config.workspace.telemetry {
277+
if config.workspace.anonymous_telemetry {
278278
telemetry::send_event(
279279
telemetry::EventType::VersionBump,
280280
Some(&pkg.name),
@@ -459,7 +459,7 @@ fn run_release_logic(root: &Path, config: &Config, dry_run: bool, verbose: bool)
459459
}
460460
}
461461

462-
if config.workspace.telemetry {
462+
if config.workspace.anonymous_telemetry {
463463
for (_, _, _, pkg_name, version, commit_count) in &tags_to_create {
464464
telemetry::send_event(
465465
telemetry::EventType::Release,

‎src/telemetry.rs‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,12 @@ struct EventPayload {
3838
}
3939

4040
fn is_enabled() -> bool {
41-
check_enabled(std::env::var("FERRFLOW_TELEMETRY").ok().as_deref())
41+
check_enabled(
42+
std::env::var("FERRFLOW_ANONYMOUS_TELEMETRY")
43+
.or_else(|_| std::env::var("FERRFLOW_TELEMETRY"))
44+
.ok()
45+
.as_deref(),
46+
)
4247
}
4348

4449
fn check_enabled(val: Option<&str>) -> bool {

0 commit comments

Comments
 (0)