Truncate response for embedding lines from github insted of silently failing.

[laundmo]

current behaviour

if the bot tries to embed lines from a GitHub link with a line count over 15 or more than 2000 characters it silently fails, not responding with anything.

proposed behaviour

there should be a response with at least some lines, ideally containing some notification that lines were omitted

to consider

• not responding at all to prevent people missing context because they assume the bot will show every line (raised by @MarkKoz on discord)
• rely on discord file previews instead as they would allow for viewing the relevant snippet entirely in discord
  • compatibility? Do file previews work on all devices?

Related discussion on discord

[MarkKoz]

I don't have much experience with the file previews, but they sound like the perfect solution for this feature. It gives a lot more headroom compared to the message length limit. I know they work on Desktop, Web, and Android.

[wookie184]

I don't have much experience with the file previews, but they sound like the perfect solution for this feature. It gives a lot more headroom compared to the message length limit. I know they work on Desktop, Web, and Android.

I don't have file previews on Android at the moment, tried on latest stable and also alpha build (88202). Not sure if it might be a partial rollout, or maybe older devices aren't supported?

Personally I still wouldn't mind using file previews for this, as most of the time it would be relevant I'd be on desktop and I can still click the link if I need to on mobile, although for others having a downloadable file appear could be confusing, so if other people on android/ios still don't have file previews i'd say the behaviour proposed in the issue would be better.

[MarkKoz]

I forgot that I use the beta app on Android. This isn't a high priority issue, so I would prefer to wait until file previews roll out everywhere since that seems to be like the best solution ultimately.

[wookie184]

File preview on mobile still isn't a thing, so I think we should go ahead with the originally proposed behaviour.