Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Internalize "known units" #327

Open
prusse-martin opened this issue Oct 11, 2023 · 0 comments
Open

Internalize "known units" #327

prusse-martin opened this issue Oct 11, 2023 · 0 comments

Comments

@prusse-martin
Copy link
Member

Add a internal list of "known units" instead of accepting all available in barril (or or auto cover those "unknown units" to the base unit when used in order to be compatible with the released version of alfasim).

Rationale:

Following the loosening of the dependency o barril (#326) users of the sdk could end up using a more recent version of barril (one that add new units) making it possible to create alfacases (and possibly plugins) that rely on units unknown to alfasim making the incompatible.

As of today the only place those "known units" are listed is in the documentation (like the "Available units for category ‘time’" section in the online documentation).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant