Sometimes posting a link to github stuff still leaves the embed of the user posted message, while I understand that this bot tries to remove it since it proposes it's own embeds.
Example:

This makes some of the features of this bot a feel redundant and messy at times.

We found this instance where posting a link to a file followed by a comma , it would quote line 1 of the file which is probably not what the user wanted:
eg: https://github.com/ModOrganizer2/modorganizer/blob/master/src/mainwindow.ui,

Current result:

Expected result:

It would be nice if the bot could not post the GitHub cards, or any other metadata. Just print the lines.

Thanks for the nice bot, it helps when using discord as the main development discussion platform.

I wanted to propose the addition of the filename and displayed lines in the bot message, in a more readable way than what is contained in the link.

For example here

the bot doesn't currently give any context of where the lines are from and the users need to extrapolate it from the link, which makes it more difficult at a glance.

In comparison this is how github renders the link:

Regarding how to show the information, I was thinking of something simple like this:

Where the filename is in bold and the line number/numbers are in normal text.

I work on an opensource project with a pretty sizable codebase and being able to see at a glance which file is being snipped would be very helpful.
In my opinion the repository name isn't necessary since it's usually clear from the context or from the filename itself. In case it's needed the developer can still get the info from the link, although less conveniently. Also adding that would risk in my opinion cluttering the extracted information too much, reducing it's readability at a glance.

Since many people aren't fans of lines interrupting conversations, it would be nice to be able to collapse lines using reactions

Also, the bot should probably not send messages about lines being too long (to limit spam)

I see this PR but I am not seeing the X being added to the Git the lines post

#19

E.g. https://foss.heptapod.net/pypy/pypy/-/blob/branch/py3.7/rpython/rtyper/lltypesystem/module/ll_math.py#L70

When the bot is sent a message like https://github.com/dolphingarlic/git-the-lines/blob/master/cogs/code_snippets.py#L16 #L1-L4. It show lines 1 to 4 instead of line 16

url tested with: https://gitlab.com/Mukundan314/test/-/blob/test/test/test.txt#L2

link tested with: https://gitlab.com/Mukundan314/test/-/blob/master/test%252F#L1

expected: to send a code block containing the contents of the range
got: sent a code block containing urlencoded version of filename + contents of the range

Hey, it's me again.

After a few weeks of using the bot on our development server the other developers asked me to kick it since there were too many embeds for all kinds of links.

The general consensus was that the line quoting feature is useful, while the other embeds like gists, commits and PR embeds are annoying and cumbersome. They might offer some info, but usually these links are posted to be clicked on in the first place so the embed isn't really that useful, they take too much space for what they offer ad interrupt the flow of the conversation with big bulky info.

So basically the feedback is that the line quoting feature is neat but all the other embeds make it annoying enough for people to not want it on the server.

If you remove those things and just keep the core feature of quoting lines I would be very happy so we can keep the bot, but I can understand if you don't want to cut features you have worked on.

I personally feel they would be good as optional additions, but since the bot doesn't have a database, you either get all the features or nothing, so in this case I would gravitate towards making the bot only do one thing well.

Let me know what you think