jchristgit
jchristgit
Hi @IngelaAndin @kikofernandez! When you talked about "big changes" in the documentation the last year I was wondering whether you're going to add triple-quoted strings, markdown support or something else....
Thanks for the feedback! I've rebased from master so the merge conflict is gone now. I think the test failures are unrelated. I do personally prefer these over mermaid because...
All good, I've rebased from master again!
Ohh, I see. No worries, I've rebased on `maint`. Have a nice weekend!
This has been implemented, primarily by Brandt.
The next person who sends "co""pilot" A"I" onto my review will have me send a Bengalese Tiger to their doorstep. And it will be very, very hungry.
Another one: ```elixir 1) test bulk_delete/2 does not include messages not found in the cache in the returned list (Nostrum.Cache.MessageCache.MnesiaAdditionalTest) test/nostrum/cache/message_cache/mnesia_additional_test.exs:167 Assertion with == failed code: assert [expected_message] == MessageCache.Mnesia.bulk_delete(@test_message.channel_id,...
Another one: ```elixir 1) test get_by_channel_id_and_author/5 it allows constraining the messages by timestamp (Nostrum.Cache.MessageCache.MnesiaAdditionalTest) test/nostrum/cache/message_cache/mnesia_additional_test.exs:310 Assertion with == failed code: assert [expected_message_one] == MessageCache.Mnesia.get_by_channel_and_author( @test_message.channel_id, @test_message.author.id, 0, 5_000_000_000_000 ) left:...
Thank you for the bug report. I will need a `:sys.trace` of the ratelimiter, please. This case should be covered presently already with the test I added for the last...
Thank you for the bug trace. Sadly it's missing info that I need. I see that messages to channel ..028 are being queued and not processed, but I need to...