Skip to content

Swap out norm for nimble_options#18

Merged
keathley merged 1 commit intomainfrom
drop-norm-dep
Mar 26, 2026
Merged

Swap out norm for nimble_options#18
keathley merged 1 commit intomainfrom
drop-norm-dep

Conversation

@keathley
Copy link
Copy Markdown
Member

No description provided.

Comment thread lib/mentat.ex Outdated
default: 5_000
],
ets_args: [
type: :any,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be a list?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah {:list, :any} is more correct here. I didn't notice that option at first.

Comment thread lib/mentat.ex
]
],
clock: [
type: :any,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be atom?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its not intended to be public. atm. its an atom since it correlates to a module. But I didn't want to enforce anything here since its only useful in development and testing.

@keathley keathley merged commit bb7baa2 into main Mar 26, 2026
3 checks passed
@keathley keathley deleted the drop-norm-dep branch March 26, 2026 15:31
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

Successfully merging this pull request may close these issues.

2 participants