Skip to content

Commit

Permalink
change target reason naming clarity
Browse files Browse the repository at this point in the history
  • Loading branch information
pnordahl committed Nov 26, 2024
1 parent 3bec4b1 commit 78d59b4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

### Fixed
- Issue preventing symlinked directories from working with `checkpoint update --pending`
- (internal) Naming of change target reason `uses_target` to `uses ` for clarity

## [3.5.6] - 2024-11-23

Expand Down
8 changes: 4 additions & 4 deletions src/app/analyze.rs
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@ impl PartialOrd for AnalyzedChangeTarget {
pub(crate) enum AnalyzedChangeTargetReason {
#[serde(rename = "target")]
Target,
#[serde(rename = "uses_target")]
UsesTarget,
#[serde(rename = "uses")]
Uses,
#[serde(rename = "ignores")]
Ignores,
}
Expand Down Expand Up @@ -290,7 +290,7 @@ fn analyze_change<'a>(
update_change_targets(
&mut change_targets,
&target2,
AnalyzedChangeTargetReason::UsesTarget,
AnalyzedChangeTargetReason::Uses,
);
trace!(target = &target2, "Added uses target");
} else {
Expand Down Expand Up @@ -499,7 +499,7 @@ mod tests {
},
AnalyzedChangeTarget {
path: target3.to_string(),
reason: AnalyzedChangeTargetReason::UsesTarget,
reason: AnalyzedChangeTargetReason::Uses,
},
]),
}];
Expand Down

0 comments on commit 78d59b4

Please sign in to comment.