Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Teams /start @user #6

Closed
0x4007 opened this issue Jul 2, 2024 · 25 comments · Fixed by #11
Closed

Teams /start @user #6

0x4007 opened this issue Jul 2, 2024 · 25 comments · Fixed by #11

Comments

@0x4007
Copy link
Member

0x4007 commented Jul 2, 2024

This reminds me that it would be really interesting in the future to support teams. Where I could write on a new task

/start @Keyrxng 

And we would both be assigned (and be eligible to split the assignee reward.)

I know a lot of developers work as lone rangers but it could make the experience more collaborative and exciting if we have a massive tasks marketplace and teams hunt projects to co-op on together.

Also if I am already assigned to a project and I write the above command, then it will add them as an assignee as well.

@Keyrxng
Copy link
Contributor

Keyrxng commented Jul 12, 2024

So I could start an issue as a team by saying /start @0x4007 and it'll assign us both.

Is this "team" now considered a single entity or two singles? E.g, could I then /stop and it'll remove my assignment but you would keep yours or we're both removed?


Only one person is going to own the PR which makes /stop with auto-close for PRs tricky.

I think to remove any burden from the user we:

  1. check if multiple assignees exist
  2. If they do and the person quitting is the PR owner then we do not close the PR, on the off chance that the other person is carrying on the work involved.
  3. If there is no activity on the issue then the remaining assignee should be kicked and the PR closed?

otherwise, the person still working will need to re-open the PR (which isn't a big deal but not the best UX)

@0x4007
Copy link
Member Author

0x4007 commented Jul 12, 2024

I suppose just check if the person quitting is the pull author. If so then close it. If not then don't.

Stop is only for self not both.

@Keyrxng
Copy link
Contributor

Keyrxng commented Jul 15, 2024

/start

Copy link

ubiquibot-dev bot commented Jul 15, 2024

Warning! This task was created over 13 days ago. Please confirm that this issue specification is accurate before starting.
DeadlineMon, Jul 15, 8:15 PM UTC
Registered Wallet 0xAe5D1F192013db889b1e2115A370aB133f359765
Tips:
  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.

Copy link

ubiquibot bot commented Jul 15, 2024

! Too many assigned issues, you have reached your max limit

@Keyrxng Keyrxng mentioned this issue Jul 15, 2024
@Keyrxng
Copy link
Contributor

Keyrxng commented Jul 16, 2024

Warning! This task was created over 13 days ago. Please confirm that this issue specification is accurate before starting.
Deadline Mon, Jul 15, 8:15 PM UTC
Registered Wallet 0xAe5D1F192013db889b1e2115A370aB133f359765

Tips:
  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.

Does this assignment comment need updated if it's this version of the command that's used?

@0x4007
Copy link
Member Author

0x4007 commented Jul 16, 2024

Does this assignment comment need updated if it's this version of the command that's used?

Let's handle it in another task if needed! Not sure if necessary, will know after testing the feature some time in the future.

Copy link
Contributor

ubiquity-os bot commented Aug 20, 2024

[ 100 WXDAI ]

@Keyrxng
Contributions Overview
View Contribution Count Reward
Issue Task 1 100
Issue Comment 2 0
Review Comment 15 0
Conversation Incentives
Comment Formatting Relevance Reward
So I could start an issue as a team by saying `/start @0x400…
0
content:
  p:
    count: 76
    score: 1
  code:
    count: 4
    score: 1
  ol:
    count: 77
    score: 0
  li:
    count: 77
    score: 1
wordValue: 0
formattingMultiplier: 0
0.9 -
Does this assignment comment need updated if it's this version o…
0
content:
  p:
    count: 15
    score: 1
wordValue: 0
formattingMultiplier: 0
0.6 -
Resolves #6 Requires #10 QA: https://github.com/ubq-testing/co…
0
content:
  p:
    count: 6
    score: 1
wordValue: 0
formattingMultiplier: 0
0.1 -
My mistake I just double-checked the [docs](https://docs.github.…
0
content:
  p:
    count: 44
    score: 1
  a:
    count: 1
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
I thought that was purely a preference thing since `octokit&…
0
content:
  p:
    count: 29
    score: 1
  code:
    count: 2
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
So in this async-await context would that just look like a try-c…
0
content:
  p:
    count: 54
    score: 1
  code:
    count: 3
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
@whilefoo I have placed it in `plugin.ts` wrapping the &…
0
content:
  p:
    count: 85
    score: 1
  code:
    count: 10
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
Thank you, I was scratching my head as to why it wasn't working …
0
content:
  p:
    count: 36
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
Unsure what the issue is but your suggestion is not working as e…
0
content:
  p:
    count: 47
    score: 1
  code:
    count: 1
    score: 1
  img:
    count: 1
    score: 0
wordValue: 0.2
formattingMultiplier: 0
1 -
the issue seemed to be that I wasn't throwing an `Error`…
0
content:
  p:
    count: 18
    score: 1
  code:
    count: 2
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
I know I thought this myself but when gentlementlegen mentioned …
0
content:
  p:
    count: 39
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
I want to add in case it's not clear, I mean to remove me saniti…
0
content:
  p:
    count: 30
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
escaping them breaks formatting when posted, edit this message t…
0
content:
  p:
    count: 86
    score: 1
  a:
    count: 2
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
@0x4007 Should I impose `maxConcurrentTasks` checks on a…
0
content:
  p:
    count: 14
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
given that this PR adds extends `/start` to allow assign…
0
content:
  p:
    count: 64
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
QA: https://github.com/ubq-testing/bot-ai/issues/27#issuecomment…
0
content:
  h2:
    count: 104
    score: 1
  code:
    count: 3
    score: 1
  img:
    count: 2
    score: 0
  p:
    count: 41
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -
https://github.com/orgs/community/discussions/7150 Seems you ar…
0
content:
  p:
    count: 52
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.2
formattingMultiplier: 0
1 -

[ 42.68 WXDAI ]

@0x4007
Contributions Overview
View Contribution Count Reward
Issue Specification 1 29.7
Issue Comment 2 1.48
Review Comment 4 11.5
Conversation Incentives
Comment Formatting Relevance Reward
This reminds me that it would be really interesting in the futur…
29.7
content:
  p:
    count: 97
    score: 1
  pre:
    count: 2
    score: 0
  code:
    count: 2
    score: 1
wordValue: 0.1
formattingMultiplier: 3
1 29.7
I suppose just check if the person quitting is the pull author. …
5.6
content:
  p:
    count: 28
    score: 1
wordValue: 0.2
formattingMultiplier: 1
0.1 0.56
Let's handle it in another task if needed! Not sure if necessary…
4.6
content:
  p:
    count: 23
    score: 1
wordValue: 0.2
formattingMultiplier: 1
0.2 0.92
@gentlementlegen rfc
0.2
content:
  p:
    count: 2
    score: 1
wordValue: 0.1
formattingMultiplier: 1
1 0.2
I don't really understand the full context of the question. I th…
7.2
content:
  p:
    count: 70
    score: 1
  code:
    count: 2
    score: 1
wordValue: 0.1
formattingMultiplier: 1
1 7.2
Checking each user seems correct based on the vision.
0.9
content:
  p:
    count: 9
    score: 1
wordValue: 0.1
formattingMultiplier: 1
1 0.9
If this isn't a lot of work I think just include the custom erro…
3.2
content:
  p:
    count: 32
    score: 1
wordValue: 0.1
formattingMultiplier: 1
1 3.2

[ 3.45 WXDAI ]

@gentlementlegen
Contributions Overview
View Contribution Count Reward
Review Comment 5 3.45
Conversation Incentives
Comment Formatting Relevance Reward
Good to go for me when you addressed the remaining comments.
0.275
content:
  p:
    count: 11
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.275
Probably should use `rest`
0.125
content:
  p:
    count: 4
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.125
Had a talk with @whilefoo and `rest` seems to be the mod…
0.575
content:
  p:
    count: 22
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.575
It is a bit risky to stringify the object without making sure th…
1.175
content:
  p:
    count: 44
    score: 1
  code:
    count: 3
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 1.175
If the backticks are not escaped they indeed could probably brea…
1.3
content:
  p:
    count: 51
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 1.3

[ 5.475 WXDAI ]

@whilefoo
Contributions Overview
View Contribution Count Reward
Review Comment 12 5.475
Conversation Incentives
Comment Formatting Relevance Reward
before you already checked that `assignees` is empty, no?
0.25
content:
  p:
    count: 9
    score: 1
  code:
    count: 1
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.25
isn't this code the same as in start function? then you can call…
0.475
content:
  p:
    count: 19
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.475
throw error
0.05
content:
  p:
    count: 2
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.05
```suggestion q: `is:open is:issue assi…
0.225
content:
  pre:
    count: 5
    score: 0
  code:
    count: 5
    score: 1
  p:
    count: 4
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.225
consider adding paginate because even though it will still work …
0.85
content:
  p:
    count: 34
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.85
consider throwing the error which `logger.error` returns…
0.975
content:
  p:
    count: 36
    score: 1
  code:
    count: 2
    score: 1
  a:
    count: 1
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.975
I think here would be a suitable place to add this logic https:/…
0.325
content:
  p:
    count: 13
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.325
because you awaited the function inside expect, it will return a…
0.875
content:
  p:
    count: 28
    score: 1
  code:
    count: 7
    score: 1
  pre:
    count: 6
    score: 0
wordValue: 0.1
formattingMultiplier: 0.25
1 0.875
I'm not sure but it's in the [docs](https://jestjs.io/docs/expec…
0.325
content:
  p:
    count: 12
    score: 1
  a:
    count: 1
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.325
why do we need to add ```, the diff already has that?
0.3
content:
  p:
    count: 12
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.3
I'm not completely sure but I think that's because for private r…
0.5
content:
  p:
    count: 20
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.5
I think it's fine without the messages (most orgs will have paid…
0.325
content:
  p:
    count: 13
    score: 1
wordValue: 0.1
formattingMultiplier: 0.25
1 0.325

@0x4007
Copy link
Member Author

0x4007 commented Aug 21, 2024

@Keyrxng I don't wanna forget about installing this feature so after you get it installed mark it as complete

@0x4007
Copy link
Member Author

0x4007 commented Aug 30, 2024

@Keyrxng can you install this plugin on the config? This should have been done right after merging.

@Keyrxng Keyrxng removed their assignment Aug 30, 2024
@Keyrxng
Copy link
Contributor

Keyrxng commented Aug 30, 2024

/start @0x4007

Copy link
Contributor

ubiquity-os bot commented Aug 30, 2024

! You have reached your max task limit

@Keyrxng
Copy link
Contributor

Keyrxng commented Aug 30, 2024

/start

Copy link
Contributor

ubiquity-os bot commented Aug 30, 2024

! You have reached your max task limit

@Keyrxng
Copy link
Contributor

Keyrxng commented Aug 30, 2024

@0x4007 it doesn't require any sort of enabling it's just built-in so if the worker that's deployed for start-stop-command is up to date then it should be working right now

@gentlementlegen can you confirm, pretty sure it's active right now?

@0x4007
Copy link
Member Author

0x4007 commented Aug 30, 2024

/start @Keyrxng

Copy link
Contributor

ubiquity-os bot commented Aug 30, 2024

! Keyrxng has reached their max task limit

@0x4007
Copy link
Member Author

0x4007 commented Aug 30, 2024

/start @gentlementlegen

Copy link
Contributor

ubiquity-os bot commented Aug 30, 2024

! gentlementlegen has reached their max task limit

@0x4007
Copy link
Member Author

0x4007 commented Aug 30, 2024

You guys need to focus on less tasks and get them done, probably!

@gentlementlegen
Copy link
Member

gentlementlegen commented Aug 30, 2024

@0x4007 I think the max is 3 currently, regardless of collaborator / contributor so pretty easy to hit (I have 3 huge tasks ongoing for me).
Also installing this feature just means merging into main to deploy the latest version of the worker (which is already done).

@0x4007
Copy link
Member Author

0x4007 commented Aug 31, 2024

Time to lock in on three or less and complete them!

@Keyrxng
Copy link
Contributor

Keyrxng commented Aug 31, 2024

@0x4007 are you happy to close this now aye?

@0x4007 0x4007 closed this as completed Aug 31, 2024
Copy link
Contributor

ubiquity-os bot commented Aug 31, 2024

[ 15 WXDAI ]

@Keyrxng
Contributions Overview
View Contribution Count Reward
Issue Comment 4 15
Review Comment 15 0
Conversation Incentives
Comment Formatting Relevance Reward
So I could start an issue as a team by saying `/start @0x400…
12.4
content:
  p:
    symbols:
      \b\w+\b:
        count: 81
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
  ol:
    symbols:
      \b\w+\b:
        count: 79
        multiplier: 0.1
    score: 0
  li:
    symbols:
      \b\w+\b:
        count: 42
        multiplier: 0.1
    score: 1
multiplier: 1
1 12.4
Does this assignment comment need updated if it's this version o…
1.7
content:
  p:
    symbols:
      \b\w+\b:
        count: 17
        multiplier: 0.1
    score: 1
multiplier: 1
0.3 0.51
@0x4007 it doesn't require any sort of enabling it's just built-…
5
content:
  p:
    symbols:
      \b\w+\b:
        count: 47
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 3
        multiplier: 0.1
    score: 1
multiplier: 1
0.4 2
@0x4007 are you happy to close this now aye?
0.9
content:
  p:
    symbols:
      \b\w+\b:
        count: 9
        multiplier: 0.1
    score: 1
multiplier: 1
0.1 0.09
Resolves #6 Requires #10 QA: https://github.com/ubq-testing/co…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 17
        multiplier: 0
    score: 1
multiplier: 0
0.2 -
My mistake I just double-checked the [docs](https://docs.github.…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 46
        multiplier: 0.2
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
I thought that was purely a preference thing since `octokit&…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 29
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
So in this async-await context would that just look like a try-c…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 59
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
@whilefoo I have placed it in `plugin.ts` wrapping the &…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 92
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
Thank you, I was scratching my head as to why it wasn't working …
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 37
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
Unsure what the issue is but your suggestion is not working as e…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 50
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
  img:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 0
multiplier: 0
1 -
the issue seemed to be that I wasn't throwing an `Error`…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 20
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
I know I thought this myself but when gentlementlegen mentioned …
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 40
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
I want to add in case it's not clear, I mean to remove me saniti…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 32
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
escaping them breaks formatting when posted, edit this message t…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 82
        multiplier: 0.2
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
@0x4007 Should I impose `maxConcurrentTasks` checks on a…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 14
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
given that this PR adds extends `/start` to allow assign…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 64
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
QA: https://github.com/ubq-testing/bot-ai/issues/27#issuecomment…
0
content:
  h2:
    symbols:
      \b\w+\b:
        count: 121
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
  img:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 0
  p:
    symbols:
      \b\w+\b:
        count: 44
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
https://github.com/orgs/community/discussions/7150 Seems you ar…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 61
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -

[ 39.64 WXDAI ]

@0x4007
Contributions Overview
View Contribution Count Reward
Issue Specification 1 25.5
Issue Comment 6 2.54
Review Comment 4 11.6
Conversation Incentives
Comment Formatting Relevance Reward
This reminds me that it would be really interesting in the futur…
25.5
content:
  p:
    symbols:
      \b\w+\b:
        count: 75
        multiplier: 0.1
    score: 1
  pre:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 0
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 5
multiplier: 3
1 25.5
I suppose just check if the person quitting is the pull author. …
5.8
content:
  p:
    symbols:
      \b\w+\b:
        count: 29
        multiplier: 0.2
    score: 1
multiplier: 1
0.1 0.58
Let's handle it in another task if needed! Not sure if necessary…
4.8
content:
  p:
    symbols:
      \b\w+\b:
        count: 24
        multiplier: 0.2
    score: 1
multiplier: 1
0.1 0.48
@Keyrxng I don't wanna forget about installing this feature so a…
4
content:
  p:
    symbols:
      \b\w+\b:
        count: 20
        multiplier: 0.2
    score: 1
multiplier: 1
0.2 0.8
@Keyrxng can you install this plugin on the config? This should …
3.4
content:
  p:
    symbols:
      \b\w+\b:
        count: 17
        multiplier: 0.2
    score: 1
multiplier: 1
0.2 0.68
You guys need to focus on less tasks and get them done, probably!
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 13
        multiplier: 0.2
    score: 1
multiplier: 1
- -
Time to lock in on three or less and complete them!
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.2
    score: 1
multiplier: 1
- -
@gentlementlegen rfc
0.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 1
multiplier: 1
1 0.2
I don't really understand the full context of the question. I th…
7.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 71
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 7.2
Checking each user seems correct based on the vision.
0.9
content:
  p:
    symbols:
      \b\w+\b:
        count: 9
        multiplier: 0.1
    score: 1
multiplier: 1
1 0.9
If this isn't a lot of work I think just include the custom erro…
3.3
content:
  p:
    symbols:
      \b\w+\b:
        count: 33
        multiplier: 0.1
    score: 1
multiplier: 1
1 3.3

[ 14.24 WXDAI ]

@gentlementlegen
Contributions Overview
View Contribution Count Reward
Issue Comment 1 0.94
Review Comment 5 13.3
Conversation Incentives
Comment Formatting Relevance Reward
@0x4007 I think the max is 3 currently, regardless of collaborat…
4.7
content:
  p:
    symbols:
      \b\w+\b:
        count: 46
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
0.2 0.94
Good to go for me when you addressed the remaining comments.
1.1
content:
  p:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 1
multiplier: 1
1 1.1
Probably should use `rest`
0.5
content:
  p:
    symbols:
      \b\w+\b:
        count: 4
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 0.5
Had a talk with @whilefoo and `rest` seems to be the mod…
2.3
content:
  p:
    symbols:
      \b\w+\b:
        count: 22
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 2.3
It is a bit risky to stringify the object without making sure th…
4.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 41
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 4.2
If the backticks are not escaped they indeed could probably brea…
5.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 51
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 5.2

[ 6.25 WXDAI ]

@whilefoo
Contributions Overview
View Contribution Count Reward
Review Comment 12 6.25
Conversation Incentives
Comment Formatting Relevance Reward
before you already checked that `assignees` is empty, no?
0.25
content:
  p:
    symbols:
      \b\w+\b:
        count: 9
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.25
isn't this code the same as in start function? then you can call…
0.5
content:
  p:
    symbols:
      \b\w+\b:
        count: 20
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.5
throw error
0.05
content:
  p:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.05
```suggestion q: `is:open is:issue assi…
0.4
content:
  pre:
    symbols:
      \b\w+\b:
        count: 12
        multiplier: 0.1
    score: 0
  code:
    symbols:
      \b\w+\b:
        count: 12
        multiplier: 0.1
    score: 1
  p:
    symbols:
      \b\w+\b:
        count: 4
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.4
consider adding paginate because even though it will still work …
0.85
content:
  p:
    symbols:
      \b\w+\b:
        count: 34
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.85
consider throwing the error which `logger.error` returns…
1.05
content:
  p:
    symbols:
      \b\w+\b:
        count: 39
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 1.05
I think here would be a suitable place to add this logic https:/…
0.625
content:
  p:
    symbols:
      \b\w+\b:
        count: 25
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.625
because you awaited the function inside expect, it will return a…
0.975
content:
  p:
    symbols:
      \b\w+\b:
        count: 28
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 1
  pre:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 0
multiplier: 0.25
1 0.975
I'm not sure but it's in the [docs](https://jestjs.io/docs/expec…
0.375
content:
  p:
    symbols:
      \b\w+\b:
        count: 14
        multiplier: 0.1
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.375
why do we need to add ```, the diff already has that?
0.275
content:
  p:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.275
I'm not completely sure but I think that's because for private r…
0.55
content:
  p:
    symbols:
      \b\w+\b:
        count: 22
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.55
I think it's fine without the messages (most orgs will have paid…
0.35
content:
  p:
    symbols:
      \b\w+\b:
        count: 14
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.35

@0x4007 0x4007 reopened this Aug 31, 2024
@0x4007 0x4007 closed this as completed Aug 31, 2024
Copy link
Contributor

ubiquity-os bot commented Aug 31, 2024

[ 100 WXDAI ]

@Keyrxng
Contributions Overview
View Contribution Count Reward
Issue Task 1 100
Issue Comment 4 0
Review Comment 15 0
Conversation Incentives
Comment Formatting Relevance Reward
So I could start an issue as a team by saying `/start @0x400…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 81
        multiplier: 0
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0
    score: 1
  ol:
    symbols:
      \b\w+\b:
        count: 79
        multiplier: 0
    score: 0
  li:
    symbols:
      \b\w+\b:
        count: 42
        multiplier: 0
    score: 1
multiplier: 0
1 -
Does this assignment comment need updated if it's this version o…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 17
        multiplier: 0
    score: 1
multiplier: 0
0.4 -
@0x4007 it doesn't require any sort of enabling it's just built-…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 47
        multiplier: 0
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 3
        multiplier: 0
    score: 1
multiplier: 0
0.3 -
@0x4007 are you happy to close this now aye?
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 9
        multiplier: 0
    score: 1
multiplier: 0
0.1 -
Resolves #6 Requires #10 QA: https://github.com/ubq-testing/co…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 17
        multiplier: 0
    score: 1
multiplier: 0
0.1 -
My mistake I just double-checked the [docs](https://docs.github.…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 46
        multiplier: 0.2
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
I thought that was purely a preference thing since `octokit&…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 29
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
So in this async-await context would that just look like a try-c…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 59
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
@whilefoo I have placed it in `plugin.ts` wrapping the &…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 92
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
Thank you, I was scratching my head as to why it wasn't working …
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 37
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
Unsure what the issue is but your suggestion is not working as e…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 50
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
  img:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 0
multiplier: 0
1 -
the issue seemed to be that I wasn't throwing an `Error`…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 20
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
I know I thought this myself but when gentlementlegen mentioned …
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 40
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
I want to add in case it's not clear, I mean to remove me saniti…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 32
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
escaping them breaks formatting when posted, edit this message t…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 82
        multiplier: 0.2
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
@0x4007 Should I impose `maxConcurrentTasks` checks on a…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 14
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
given that this PR adds extends `/start` to allow assign…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 64
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
QA: https://github.com/ubq-testing/bot-ai/issues/27#issuecomment…
0
content:
  h2:
    symbols:
      \b\w+\b:
        count: 121
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
  img:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.2
    score: 0
  p:
    symbols:
      \b\w+\b:
        count: 44
        multiplier: 0.2
    score: 1
multiplier: 0
1 -
https://github.com/orgs/community/discussions/7150 Seems you ar…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 61
        multiplier: 0.2
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.2
    score: 1
multiplier: 0
1 -

[ 38.74 WXDAI ]

@0x4007
Contributions Overview
View Contribution Count Reward
Issue Specification 1 25.5
Issue Comment 6 1.64
Review Comment 4 11.6
Conversation Incentives
Comment Formatting Relevance Reward
This reminds me that it would be really interesting in the futur…
25.5
content:
  p:
    symbols:
      \b\w+\b:
        count: 75
        multiplier: 0.1
    score: 1
  pre:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 0
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 5
multiplier: 3
1 25.5
I suppose just check if the person quitting is the pull author. …
5.8
content:
  p:
    symbols:
      \b\w+\b:
        count: 29
        multiplier: 0.2
    score: 1
multiplier: 1
0.2 1.16
Let's handle it in another task if needed! Not sure if necessary…
4.8
content:
  p:
    symbols:
      \b\w+\b:
        count: 24
        multiplier: 0.2
    score: 1
multiplier: 1
0.1 0.48
@Keyrxng I don't wanna forget about installing this feature so a…
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 20
        multiplier: 0.2
    score: 1
multiplier: 1
- -
@Keyrxng can you install this plugin on the config? This should …
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 17
        multiplier: 0.2
    score: 1
multiplier: 1
- -
You guys need to focus on less tasks and get them done, probably!
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 13
        multiplier: 0.2
    score: 1
multiplier: 1
- -
Time to lock in on three or less and complete them!
0
content:
  p:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.2
    score: 1
multiplier: 1
- -
@gentlementlegen rfc
0.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 1
multiplier: 1
1 0.2
I don't really understand the full context of the question. I th…
7.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 71
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 7.2
Checking each user seems correct based on the vision.
0.9
content:
  p:
    symbols:
      \b\w+\b:
        count: 9
        multiplier: 0.1
    score: 1
multiplier: 1
1 0.9
If this isn't a lot of work I think just include the custom erro…
3.3
content:
  p:
    symbols:
      \b\w+\b:
        count: 33
        multiplier: 0.1
    score: 1
multiplier: 1
1 3.3

[ 14.24 WXDAI ]

@gentlementlegen
Contributions Overview
View Contribution Count Reward
Issue Comment 1 0.94
Review Comment 5 13.3
Conversation Incentives
Comment Formatting Relevance Reward
@0x4007 I think the max is 3 currently, regardless of collaborat…
4.7
content:
  p:
    symbols:
      \b\w+\b:
        count: 46
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
0.2 0.94
Good to go for me when you addressed the remaining comments.
1.1
content:
  p:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 1
multiplier: 1
1 1.1
Probably should use `rest`
0.5
content:
  p:
    symbols:
      \b\w+\b:
        count: 4
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 0.5
Had a talk with @whilefoo and `rest` seems to be the mod…
2.3
content:
  p:
    symbols:
      \b\w+\b:
        count: 22
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 2.3
It is a bit risky to stringify the object without making sure th…
4.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 41
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 4.2
If the backticks are not escaped they indeed could probably brea…
5.2
content:
  p:
    symbols:
      \b\w+\b:
        count: 51
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 1
1 5.2

[ 6.25 WXDAI ]

@whilefoo
Contributions Overview
View Contribution Count Reward
Review Comment 12 6.25
Conversation Incentives
Comment Formatting Relevance Reward
before you already checked that `assignees` is empty, no?
0.25
content:
  p:
    symbols:
      \b\w+\b:
        count: 9
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.25
isn't this code the same as in start function? then you can call…
0.5
content:
  p:
    symbols:
      \b\w+\b:
        count: 20
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.5
throw error
0.05
content:
  p:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.05
```suggestion q: `is:open is:issue assi…
0.4
content:
  pre:
    symbols:
      \b\w+\b:
        count: 12
        multiplier: 0.1
    score: 0
  code:
    symbols:
      \b\w+\b:
        count: 12
        multiplier: 0.1
    score: 1
  p:
    symbols:
      \b\w+\b:
        count: 4
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.4
consider adding paginate because even though it will still work …
0.85
content:
  p:
    symbols:
      \b\w+\b:
        count: 34
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.85
consider throwing the error which `logger.error` returns…
1.05
content:
  p:
    symbols:
      \b\w+\b:
        count: 39
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 2
        multiplier: 0.1
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 1.05
I think here would be a suitable place to add this logic https:/…
0.625
content:
  p:
    symbols:
      \b\w+\b:
        count: 25
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.625
because you awaited the function inside expect, it will return a…
0.975
content:
  p:
    symbols:
      \b\w+\b:
        count: 28
        multiplier: 0.1
    score: 1
  code:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 1
  pre:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 0
multiplier: 0.25
1 0.975
I'm not sure but it's in the [docs](https://jestjs.io/docs/expec…
0.375
content:
  p:
    symbols:
      \b\w+\b:
        count: 14
        multiplier: 0.1
    score: 1
  a:
    symbols:
      \b\w+\b:
        count: 1
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.375
why do we need to add ```, the diff already has that?
0.275
content:
  p:
    symbols:
      \b\w+\b:
        count: 11
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.275
I'm not completely sure but I think that's because for private r…
0.55
content:
  p:
    symbols:
      \b\w+\b:
        count: 22
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.55
I think it's fine without the messages (most orgs will have paid…
0.35
content:
  p:
    symbols:
      \b\w+\b:
        count: 14
        multiplier: 0.1
    score: 1
multiplier: 0.25
1 0.35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants