[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ELPA] New package: jami-bot and org-jami-bot
From: |
Philip Kaludercic |
Subject: |
Re: [ELPA] New package: jami-bot and org-jami-bot |
Date: |
Tue, 09 Jan 2024 19:39:35 +0000 |
Stefan Kangas <stefankangas@gmail.com> writes:
> Hanno Perrey <hanno@hoowl.se> writes:
>
>> I would like to propose two tightly-coupled packages for inclusion into
>> ELPA: jami-bot and org-jami-bot.
>>
>> jami-bot is an extendable chat bot for the distributed, private
>> messenger Jami. It interacts with the locally-installed Jami daemon via
>> D-Bus and reacts to both plain text messages and file transfers sent to
>> local accounts.
>>
>> org-jami-bot builds upon jami-bot and extends it with Org mode capture
>> functionality for text messages and images. It allows to schedule agenda
>> items at specific dates, compose multi-measure captures and capture
>> images -- all by sending a message via the GNU Jami messenger.
>>
>> I would like to keep maintaining both packages at their current
>> repositories:
>>
>> - https://gitlab.com/hperrey/jami-bot
>> - https://gitlab.com/hperrey/org-jami-bot
>>
>> I have signed the copyright paperwork with the FSF already and changed
>> the copyright in the respective files for both packages.
>>
>> As this is the first time I am proposing a package for ELPA, any
>> guidance on the process and feedback would be highly welcome!
>
> Philip, could you please help add these packages to GNU ELPA?
>
> The copyright assignment is all taken care of.
Sure, but first Hanno could you bump the version header, so that the
ELPA build system uses a newer commit, that includes the stylistic
changes + the updated copyright line?
A few comments from my side:
- It seems like a few variable (`jami-bot-account-user-names'?) would be
nice to have as proper user options, defined with `defcustom' and a
type.
- Try to sharp-quote (#') function symbols.
- If possible, try to join file paths using functions like
`expand-file-name' with `default-directory' set, instead of using
"low-level" string-manipulation.
- There seem to be some indentation issues (jami-bot-send-message), try
running `indent-region' on the entire buffer.
- There are a few minor checkdoc issues, consider resolving them.
Re: [ELPA] New package: jami-bot and org-jami-bot,
Philip Kaludercic <=
- Re: [ELPA] New package: jami-bot and org-jami-bot, Richard Stallman, 2024/01/11
- Re: [ELPA] New package: jami-bot and org-jami-bot, Stefan Kangas, 2024/01/11
- Re: [ELPA] New package: jami-bot and org-jami-bot, Eli Zaretskii, 2024/01/12
- Re: [ELPA] New package: jami-bot and org-jami-bot, Ihor Radchenko, 2024/01/12
- Re: [ELPA] New package: jami-bot and org-jami-bot, Eli Zaretskii, 2024/01/12
- Re: [ELPA] New package: jami-bot and org-jami-bot, Ihor Radchenko, 2024/01/12
- Re: [ELPA] New package: jami-bot and org-jami-bot, Eli Zaretskii, 2024/01/12
- Re: [ELPA] New package: jami-bot and org-jami-bot, Ihor Radchenko, 2024/01/12
- Re: [ELPA] New package: jami-bot and org-jami-bot, Eli Zaretskii, 2024/01/12
- Re: [ELPA] New package: jami-bot and org-jami-bot, Ihor Radchenko, 2024/01/12