-
-
Notifications
You must be signed in to change notification settings - Fork 43
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
Remove usage of async-timer
#301
Comments
I see this has already been addressed here: #291 (comment). @mikonieminen Sorry for tagging you. Were you able to solve this problem? |
This is due to issue: geofmureithi/apalis#301
Can confirm that downgrading to |
* fix(frontend): open correct modal * refactor(frontend): change name of variables * build: remove changes to build profile * build(backend): add `cached` deps * feat(backend): start using `cached` for caching seen * build(backend): add features for `cached` * fix(backend): use `cached` for deps * build(backend): remove `retainer` deps * build(backend): bump version * refactor(backend): move trimming stuff to dedicated func * fix(backend): add logging when getting intgerations data * build(backend): downgrade apalis This is due to issue: geofmureithi/apalis#301
@IgnisDa I have my separate branch that replaces use of I haven't tried if the the older version would solve the problem. You can still find the branch from: https://github.com/mikonieminen/apalis/tree/fix/async-timer-causing-panics |
@geofmureithi Any updates on this? |
@IgnisDa Will be releasing version 0.6 alpha over the weekend. |
@geofmureithi Could you cut a new release? |
Would it be possible to remove usage of
async-timer
[reason]?I am getting a
BorrowedMutError
and here is the full stacktrace:It has calls to
async-timer
andapalis
is the only one which uses this crate.The text was updated successfully, but these errors were encountered: