search

ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
17 stars 60 forks source link

`ubiquibot-config-default.json` property names #742

Open 0x4007 opened 1 year ago

0x4007 commented 1 year ago

I'd like group input on this but the intent is to make the config intuitive to understand for people to read. We eventually will need to produce documentation outlining all of the bot capabilities.

I wish that the property name was more clear. When I saw this in ubiquity/ubiquibot-config/.github/ubiquibot-config.yml I had to trace this down to this pull request to understand what it does.

Perhaps we can rename to: 

  "public-access-control": {
     "set-label": false,
     "fund-external-closed-issue": false

What do you guys think?

Alternatively we can move it to a separate section. Some type of grouping related to payment permit generation instead of batching it together with outsiders being about to set labels.

_Originally posted by @pavlovcik in https://github.com/ubiquity/ubiquibot/pull/697#discussion_r1320831528_

AlooDon commented 10 months ago

/start

ubiquibot[bot] commented 10 months ago

Skipping /start because it is disabled on this repo

PhantomCracker commented 9 months ago

@pavlovcik can we still work on bot issues or are we still in refactorization?

0x4007 commented 9 months ago

still in refactorization

I think I might have already handled this issue though. I'll check them all off once its all merged in.