I'm on discord servers - Sidekick is offline on one, but online on the other.

Right! But keep in mind that the larget the PR is the harder it is to review and accept D

   Compiling inlinable_string v0.1.11 (C:\Users\artem\Downloads\qwe\inlinable_string)
error: unexpected closing delimiter: `}`
   --> src\inline_string.rs:505:5
    |
275 | impl InlineString {
    |                   - this opening brace...
...
504 |         }
    |         - ...matches this closing brace
505 |     }
    |     ^ unexpected closing delimiter

error: aborting due to previous error

error: could not compile `inlinable_string`.

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: unexpected closing delimiter: `}`
   --> src\inline_string.rs:505:5
    |
275 | impl InlineString {
    |                   - this opening brace...
...
504 |         }
    |         - ...matches this closing brace
505 |     }
    |     ^ unexpected closing delimiter

error: aborting due to previous error

error: build failed

Without RangeBounds there's no replace_range and remove_range

Can't the generics live on another trait?

Note that if you have a code sample in TypeScript then I can try to reuse it in #160 (thinking of making an open-source version of a TypeScript-based pluggable dice engine for people to play with)

Thus there gonna be only two breaking changes: insert_str and drain

Can't the drain be a part of a different trait?

split_off is a tricky one; should it return a String or an instance of Self?

Maybe two methods, one returning a String to more easily fit the String-based APIs and another, with an extra suffix in the method name, returning InlinableString for the sake of speed?

Hey! Thanks for the patch! After a couple of months I finally have a little moment to look at what's going on here D New methods are cool. If you would be so kind as to split it into smaller patches, one per new method perhaps, and start with the patches that retain the API and binary compatibility, I'm sure I'll find the time to check and then merge them. As it is, I see some hard decisions here, like how much the usefulness of a trait object and maybe the added link time overhead due to more generics compare to the usefulness of new methods and the strain on developers from having an extra version to be aware of, and I would prefer to simply avoid making those.

Hi,

since around 20 minutes the response of Sidekick is delayed by about 5-7 minutes.

Any chance I can fix it by myself? I read you were trying giving shard restart to the users in 2018.

Kind regards

#232

After the cloudflare outage today on July 17 2020, Sidekick has been accepting every command in duplicate, outputting double results in return. I.e. a single /r 1d100 command results in two separate 1d100 rolls by the bot. I tried kicking and re-inviting the bot, but that did not solve the issue. I quit and restarted discord, nothing. It happens to every person on my server and in every channel it is in. I have no idea how to fix this on my end.

BTW, never heard of kicking and reinviting the bot to fix anything.

This project is not open source.

Noticed that Chromium stays around in the process list even if the browser window is closed. Watching for the page closing event helps to workaround this, e.g.

import {chromium} from 'playwright';
(async () => {
  const browser = await chromium.launch ({headless: false});
  const context = await browser.newContext();
  const page = await context.newPage();
  page.on ('close', () => {
    console.log ('page closed');
    browser.close();});
})();

When you see strings used for prices, it is often done for the reason of avoiding the floating point errors and giving the users a choice in what kind of arithmetics they want to use (some users might want to parse directly into BigRational)