-
Notifications
You must be signed in to change notification settings - Fork 346
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
TimeFunc being static prevents from using this package in a parallel test #188
Comments
Thanks for opening this issue and linking to the PR. I suggest keeping it open (maybe adding a /v5 label) so we make sure to consider it if/when thinking through what a future release would look like. |
Did you by any chance have a look at the |
Fixed by #234 |
I tried to bring this
TimeFunc
on theParserOptions
, then pass it in theClaims
But I just saw a recent retraction of something very similar.
#184
So, I would believe you will not accept a PR like mine until v5, but having the
TimeFunc()
local makes more sense for testingjsgoupil@a6c0c03
I hope you can consider this bug for a future release.
The text was updated successfully, but these errors were encountered: