search

s-kostyaev / ellama

Ellama is a tool for interacting with large language models from Emacs.
GNU General Public License v3.0
485 stars 30 forks source link

Make it possible to customize paragraph filling behavior #41

Closed Stebalien closed 9 months ago

Stebalien commented 9 months ago

I've switched from checking for "not prog-mode" to "is text-mode", but that should work in all reasonable cases.

Built on #40

s-kostyaev commented 9 months ago

@Stebalien rebase it, please. Also, why you mention paragraphs? We filling region here. It can contain multiple paragraphs. And for me customization variable name looks odd. Maybe it's just me.

Stebalien commented 9 months ago

Also, why you mention paragraphs? We filling region here. It can contain multiple paragraphs. And for me customization variable name looks odd. Maybe it's just me.

Well, we're filling all paragraphs in text produced by the LLM. I don't really want to use the word "region" because we're not filling a user-specified region.

E.g., from the text of fill-region:

Fill each of the paragraphs in the region.

Looking through similar variables:

If paragraphs is confusing, I can change it to fill-text.