You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Only one file requires the usage of Fractions and the implementation relies on it without a clear purpose; the numeric computations can all be worked out easily without relying on fractions. Not to mention other unreasonable usages of BigInteger. The file that uses this package is this. Additionally, the package is explicitly referenced in two other projects without a real usage.
This is a leaking package reference that serves no purpose in most applications, and opinionates the usage of a library that has poor performance for real-life implementations using numeric fractions. Minimizing package references is very helpful for consumers to reduce build times and package cache sizes.
The text was updated successfully, but these errors were encountered:
Only one file requires the usage of
Fractions
and the implementation relies on it without a clear purpose; the numeric computations can all be worked out easily without relying on fractions. Not to mention other unreasonable usages ofBigInteger
. The file that uses this package is this. Additionally, the package is explicitly referenced in two other projects without a real usage.This is a leaking package reference that serves no purpose in most applications, and opinionates the usage of a library that has poor performance for real-life implementations using numeric fractions. Minimizing package references is very helpful for consumers to reduce build times and package cache sizes.
The text was updated successfully, but these errors were encountered: