tildeverse
/
rfcs
4
6
Fork 6
Code Issues 5 Pull Requests 1 Releases Wiki Activity

Standardise IRC bot behavior #2

Merged
khuxkm merged 7 commits from draft-ircbots into master 2018-09-26 20:26:09 +00:00
Conversation 14 Commits 7 Files Changed 1 +43
khuxkm commented 2018-08-31 02:52:22 +00:00
Owner
Copy Link

Basically just codifies the IRC bots wiki page.

Basically just codifies the IRC bots wiki page.
ben commented 2018-08-31 02:55:02 +00:00
Owner
Copy Link

I'd like to require bots to set mode +B upon joining. What do you think?

I'd like to require bots to set mode +B upon joining. What do you think?
BrendanTCC commented 2018-08-31 03:19:02 +00:00
First-time contributor
Copy Link

you might want to check that +B doesn't already exist, Ben. :)

you might want to check that +B doesn't already exist, Ben. :)
ben commented 2018-08-31 03:23:55 +00:00
Owner
Copy Link

@brendantcc sorry what?

usermode +B is meant for bots.

@brendantcc sorry what? usermode +B is meant for bots.
BrendanTCC commented 2018-08-31 03:27:57 +00:00
First-time contributor
Copy Link

it is?

it is?
slipyx commented 2018-08-31 06:05:42 +00:00
First-time contributor
Copy Link

So this assumes all bots use '!' as their command prefix?

So this assumes all bots use '!' as their command prefix?
khuxkm commented 2018-08-31 06:06:39 +00:00
Author
Owner
Copy Link

No. However, for this one usecase, the bot must respond to it with the ! prefix.

No. However, for this one usecase, the bot must respond to it with the ! prefix.
khuxkm commented 2018-09-02 15:44:45 +00:00
Author
Owner
Copy Link

@ben ping
What do you think? Is this good?

@ben ping What do you think? Is this good?
ben commented 2018-09-02 16:03:17 +00:00
Owner
Copy Link

lgtm. can you update https://tilde.chat/wiki/?page=bots as well?

lgtm. can you update https://tilde.chat/wiki/?page=bots as well?
khuxkm commented 2018-09-07 18:53:36 +00:00
Author
Owner
Copy Link

I'm gonna get this ready for merge. Assigning RFC # 2.

I'm gonna get this ready for merge. Assigning RFC # 2.
ben commented 2018-09-07 19:03:07 +00:00
Owner
Copy Link

I'm not sure that you need to specify the ~town stuff, since it's clearly stated above.

I'm not sure that you need to specify the ~town stuff, since it's clearly stated above.
khuxkm commented 2018-09-07 19:04:50 +00:00
Author
Owner
Copy Link

Still, it's worth explaining.

Still, it's worth explaining.
ben commented 2018-09-07 19:06:05 +00:00
Owner
Copy Link

Ehh ok. It'd duplicate information IMO but ok.

Ehh ok. It'd duplicate information IMO but ok.
khuxkm commented 2018-09-07 19:07:50 +00:00
Author
Owner
Copy Link

Where I reference the tilde.town standard, I'm not stating that tilde.town bots need to follow !botlist. That's why I specifically state that in the Config Considerations section. Better explicit than implicit.

Where I reference the tilde.town standard, I'm not stating that tilde.town bots need to follow `!botlist`. That's why I specifically state that in the Config Considerations section. Better explicit than implicit.
khuxkm closed this pull request 2018-09-26 20:26:09 +00:00
khuxkm deleted branch draft-ircbots 2018-09-26 20:26:14 +00:00
khuxkm commented 2018-09-26 21:00:45 +00:00
Author
Owner
Copy Link

Merged and put into action! RFC #2

Merged and put into action! [RFC #2](https://rfc.tildeverse.org/rfc.php?number=2)
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: tildeverse/rfcs#2
No description provided.
Delete Branch "draft-ircbots"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?