Skip to content

Commit

Permalink
fix: Pow and Root now ensure that zero exponents are removed
Browse files Browse the repository at this point in the history
  • Loading branch information
RebeccaStevens committed Aug 9, 2023
1 parent ae679dc commit 0929ef3
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions src/units.ts
Original file line number Diff line number Diff line change
Expand Up @@ -130,9 +130,12 @@ export type Pow<T extends number, N extends Exponent> = T extends Unit<
*
* @group Unit Subvalue Functions
*/
export type PowUnitSubvalues<T extends UnitSubvalues, E extends Exponent> = {
export type PowUnitSubvalues<
T extends UnitSubvalues,
E extends Exponent,
> = ExcludeUnitZeroSubvalues<{
[S in keyof T]: MultiplyExponents<GetExponent<T, S>, E>;
};
}>;

/**
* Take the nth root of a unit.
Expand All @@ -156,9 +159,12 @@ export type Root<T extends number, N extends Exponent> = T extends Unit<
*
* @group Unit Subvalue Functions
*/
export type RootUnitSubvalues<T extends UnitSubvalues, E extends Exponent> = {
export type RootUnitSubvalues<
T extends UnitSubvalues,
E extends Exponent,
> = ExcludeUnitZeroSubvalues<{
[S in keyof T]: DivideExponents<GetExponent<T, S>, E>;
};
}>;

// Tests
// eslint-disable-next-line eslint-comments/disable-enable-pair
Expand Down

0 comments on commit 0929ef3

Please sign in to comment.