-
Notifications
You must be signed in to change notification settings - Fork 11
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
Comments
So I could start an issue as a team by saying Is this "team" now considered a single entity or two singles? E.g, could I then Only one person is going to own the PR which makes I think to remove any burden from the user we:
otherwise, the person still working will need to re-open the PR (which isn't a big deal but not the best UX) |
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. |
/start |
Tips:
|
! Too many assigned issues, you have reached your max limit |
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. |
|
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… | 0content: 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… | 0content: p: count: 15 score: 1 wordValue: 0 formattingMultiplier: 0 | 0.6 | - |
Resolves #6 Requires #10 QA: https://github.com/ubq-testing/co… | 0content: p: count: 6 score: 1 wordValue: 0 formattingMultiplier: 0 | 0.1 | - |
My mistake I just double-checked the [docs](https://docs.github.… | 0content: 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&… | 0content: 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… | 0content: 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 &… | 0content: 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 … | 0content: 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… | 0content: 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`… | 0content: 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 … | 0content: 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… | 0content: p: count: 30 score: 1 wordValue: 0.2 formattingMultiplier: 0 | 1 | - |
escaping them breaks formatting when posted, edit this message t… | 0content: p: count: 86 score: 1 a: count: 2 score: 1 wordValue: 0.2 formattingMultiplier: 0 | 1 | - |
@0x4007 Should I impose `maxConcurrentTasks` checks on a… | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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.7content: 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.6content: 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.6content: p: count: 23 score: 1 wordValue: 0.2 formattingMultiplier: 1 | 0.2 | 0.92 |
@gentlementlegen rfc | 0.2content: 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.2content: 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.9content: 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.2content: 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.275content: p: count: 11 score: 1 wordValue: 0.1 formattingMultiplier: 0.25 | 1 | 0.275 |
Probably should use `rest` | 0.125content: 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.575content: 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.175content: 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.3content: 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.25content: 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.475content: p: count: 19 score: 1 wordValue: 0.1 formattingMultiplier: 0.25 | 1 | 0.475 |
throw error | 0.05content: p: count: 2 score: 1 wordValue: 0.1 formattingMultiplier: 0.25 | 1 | 0.05 |
```suggestion q: `is:open is:issue assi… | 0.225content: 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.85content: p: count: 34 score: 1 wordValue: 0.1 formattingMultiplier: 0.25 | 1 | 0.85 |
consider throwing the error which `logger.error` returns… | 0.975content: 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.325content: 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.875content: 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.325content: 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.3content: 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.5content: 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.325content: p: count: 13 score: 1 wordValue: 0.1 formattingMultiplier: 0.25 | 1 | 0.325 |
@Keyrxng I don't wanna forget about installing this feature so after you get it installed mark it as complete |
@Keyrxng can you install this plugin on the config? This should have been done right after merging. |
/start @0x4007 |
! You have reached your max task limit |
/start |
! You have reached your max task limit |
@0x4007 it doesn't require any sort of enabling it's just built-in so if the worker that's deployed for @gentlementlegen can you confirm, pretty sure it's active right now? |
/start @Keyrxng |
! Keyrxng has reached their max task limit |
/start @gentlementlegen |
! gentlementlegen has reached their max task limit |
You guys need to focus on less tasks and get them done, probably! |
@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). |
Time to lock in on three or less and complete them! |
@0x4007 are you happy to close this now aye? |
|
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.4content: 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.7content: 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-… | 5content: 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.9content: 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… | 0content: 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.… | 0content: 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&… | 0content: 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… | 0content: 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 &… | 0content: 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 … | 0content: 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… | 0content: 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`… | 0content: 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 … | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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.5content: 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.8content: 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.8content: 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… | 4content: 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.4content: 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! | 0content: 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! | 0content: p: symbols: \b\w+\b: count: 11 multiplier: 0.2 score: 1 multiplier: 1 | - | - |
@gentlementlegen rfc | 0.2content: 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.2content: 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.9content: 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.3content: 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.7content: 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.1content: p: symbols: \b\w+\b: count: 11 multiplier: 0.1 score: 1 multiplier: 1 | 1 | 1.1 |
Probably should use `rest` | 0.5content: 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.3content: 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.2content: 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.2content: 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.25content: 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.5content: p: symbols: \b\w+\b: count: 20 multiplier: 0.1 score: 1 multiplier: 0.25 | 1 | 0.5 |
throw error | 0.05content: 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.4content: 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.85content: 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.05content: 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.625content: 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.975content: 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.375content: 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.275content: 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.55content: 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.35content: p: symbols: \b\w+\b: count: 14 multiplier: 0.1 score: 1 multiplier: 0.25 | 1 | 0.35 |
|
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… | 0content: 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… | 0content: 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-… | 0content: 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? | 0content: 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… | 0content: 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.… | 0content: 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&… | 0content: 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… | 0content: 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 &… | 0content: 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 … | 0content: 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… | 0content: 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`… | 0content: 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 … | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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… | 0content: 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.5content: 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.8content: 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.8content: 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… | 0content: 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 … | 0content: 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! | 0content: 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! | 0content: p: symbols: \b\w+\b: count: 11 multiplier: 0.2 score: 1 multiplier: 1 | - | - |
@gentlementlegen rfc | 0.2content: 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.2content: 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.9content: 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.3content: 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.7content: 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.1content: p: symbols: \b\w+\b: count: 11 multiplier: 0.1 score: 1 multiplier: 1 | 1 | 1.1 |
Probably should use `rest` | 0.5content: 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.3content: 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.2content: 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.2content: 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.25content: 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.5content: p: symbols: \b\w+\b: count: 20 multiplier: 0.1 score: 1 multiplier: 0.25 | 1 | 0.5 |
throw error | 0.05content: 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.4content: 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.85content: 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.05content: 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.625content: 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.975content: 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.375content: 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.275content: 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.55content: 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.35content: p: symbols: \b\w+\b: count: 14 multiplier: 0.1 score: 1 multiplier: 0.25 | 1 | 0.35 |
This reminds me that it would be really interesting in the future to support teams. Where I could write on a new task
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.
The text was updated successfully, but these errors were encountered: