Change logs to use ubiquity-dao/ubiquibot-logger
#4
Assignees
Comments
|
/wallet 0xFaF2BD96905b7DAf432534FF4Fa75D90A36614eE |
+ Successfully registered wallet address |
+ Successfully registered wallet address |
|
/start |
Tips:
|
Tips:
|
|
Since there is |
|
@wolfmic I think using |
|
/START |
|
/start |
Tip
|
|
/start |
Tip
|
Tip
|
|
@gentlementlegen @whilefoo can one bot be disabled now? Do we just need to empty out one of the configs? Let's make @ubiquity use production, and @ubiquity-os, @ubiquity-os-marketplace use development? |
|
Marketplace needs both otherwise the runs won't work, you will get the |
0x4007
added a commit
to ubiquity-os-marketplace/.ubiquity-os
that referenced
this issue
Oct 17, 2024
0x4007
added a commit
to ubiquity-os/.ubiquity-os
that referenced
this issue
Oct 17, 2024
|
/help |
Available Commands
|
Available Commands
|
! You are already assigned to this task. |
! All linked pull requests must be closed to generate rewards. |
|
| View | Contribution | Count | Reward |
|---|---|---|---|
| Issue | Task | 1 | 50 |
| Review | Comment | 4 | 0 |
Conversation Incentives
| Comment | Formatting | Relevance | Reward |
|---|---|---|---|
Resolves #4 | 0content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 2
wordValue: 0
result: 0
| 0.4 | 0 |
Sometimes the test failed because it wasnt able to resolve an EN… | 0content:
content:
p:
score: 0
elementCount: 2
result: 0
regex:
wordCount: 24
wordValue: 0
result: 0
| 0.1 | 0 |
Shall I undo this change then? | 0content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 6
wordValue: 0
result: 0
| 0.1 | 0 |
I reverted the changes and the tests also passI reverted the c… | 0content:
content:
p:
score: 0
elementCount: 2
result: 0
regex:
wordCount: 18
wordValue: 0
result: 0
| 0.2 | 0 |
[ 0.345 WXDAI ]
@wolfmic
Contributions Overview
| View | Contribution | Count | Reward |
|---|---|---|---|
| Issue | Comment | 1 | 0.345 |
Conversation Incentives
| Comment | Formatting | Relevance | Reward |
|---|---|---|---|
Since there is `context.logger.warn()` in two files (plu… | 1.7content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 28
wordValue: 0.1
result: 1.7
| 0.8 | 0.345 |
[ 14.918 WXDAI ]
@gentlementlegen
Contributions Overview
| View | Contribution | Count | Reward |
|---|---|---|---|
| Issue | Specification | 1 | 8.19 |
| Issue | Comment | 2 | 0.821 |
| Review | Comment | 9 | 5.907 |
Conversation Incentives
| Comment | Formatting | Relevance | Reward |
|---|---|---|---|
Currently the logs are handled manually with `console` m… | 2.73content:
content:
p:
score: 0
elementCount: 3
result: 0
regex:
wordCount: 49
wordValue: 0.1
result: 2.73
| 1 | 8.19 |
@wolfmic I think using `logger.error` instead of warn is… | 0.77content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 11
wordValue: 0.1
result: 0.77
| 0.7 | 0.539 |
Marketplace needs both otherwise the runs won't work, you will g… | 0.94content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 14
wordValue: 0.1
result: 0.94
| 0.3 | 0.282 |
If you use the logger you should not have to manually build thes… | 1.38content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 22
wordValue: 0.1
result: 1.38
| 0.9 | 1.242 |
Delete comments | 0.18content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 2
wordValue: 0.1
result: 0.18
| 0.3 | 0.054 |
```suggestionawait addCommentToIssue(context, … | 0.65content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 9
wordValue: 0.1
result: 0.65
| 1 | 0.65 |
```suggestion``` | 0content:
content: {}
result: 0
regex:
wordCount: 0
wordValue: 0.1
result: 0
| 0.1 | 0 |
I think it would make more sense to just use `passthrough … | 1.28content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 20
wordValue: 0.1
result: 1.28
| 0.8 | 1.024 |
Yes this happens. I guess it's fine for tests, but ideally our R… | 1.11content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 17
wordValue: 0.1
result: 1.11
| 0.6 | 0.666 |
@0x4007 What do you think would be the best approach? In the con… | 2.05content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 35
wordValue: 0.1
result: 2.05
| 0.4 | 0.82 |
@ariesgun I am okay with the changes, however the `Jest`… | 1.11content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 17
wordValue: 0.1
result: 1.11
| 0.5 | 0.555 |
QA: https://github.com/Meniole/command-wallet/issues/5#issuecomm… | 1.28content:
content:
p:
score: 0
elementCount: 2
result: 0
regex:
wordCount: 20
wordValue: 0.1
result: 1.28
| 0.7 | 0.896 |
[ 2.199 WXDAI ]
@0x4007
Contributions Overview
| View | Contribution | Count | Reward |
|---|---|---|---|
| Issue | Comment | 1 | 1.17 |
| Review | Comment | 1 | 1.029 |
Conversation Incentives
| Comment | Formatting | Relevance | Reward |
|---|---|---|---|
@gentlementlegen @whilefoo can one bot be disabled now? Do we ju… | 1.95content:
content:
p:
score: 0
elementCount: 3
result: 0
regex:
wordCount: 33
wordValue: 0.1
result: 1.95
| 0.6 | 1.17 |
I'm not sure if its valid to skip implementing this test tempora… | 3.43content:
content:
p:
score: 0
elementCount: 1
result: 0
regex:
wordCount: 64
wordValue: 0.1
result: 3.43
| 0.3 | 1.029 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the logs are handled manually with
consolemessages. They should use our logger package to follow our usual display standards and logic.https://github.com/ubiquity/ubiquibot-logger
C.f. #2 (comment) and #2 (comment)
The text was updated successfully, but these errors were encountered: