-
Notifications
You must be signed in to change notification settings - Fork 11
Conversation
@rultor merge |
@g4s8 Oops, I failed. You can see the full log here (spent 4min)
|
This pull request #411 is assigned to @victornoel/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @g4s8/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@g4s8 just one comment for now, and also, shouldn't we add @todos
to continue the work?
<user id="0crat"> | ||
<activated>2016-12-29T09:03:21.684Z</activated> | ||
</user> | ||
</users> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@g4s8 I'm not clear what this file is for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@victornoel this file is like a test resource, the build is running xsd schema against this file and checks that it's valid
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@g4s8 ah indeed, sorry I now realise the README explains that. So the work is finished for that task or do we need todos (or maybe you will be making multiple PRs?)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@victornoel yes, it's finished in this repo. It'll be continued in farm
: https://github.com/zerocracy/farm/issues/2245
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@g4s8 I understand, thank you for the explanation :)
@g4s8 ping |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@g4s8 thx for taking the time to answer my questions :)
@rultor merge |
@g4s8 Oops, I failed. You can see the full log here (spent 3min)
|
@g4s8 can we get this merged? It's taking space in my agenda :) |
@g4s8 ping |
1 similar comment
@g4s8 ping |
@g4s8 can we get this one merged please? |
@rultor merge |
@g4s8 Oops, I failed. You can see the full log here (spent 7min)
|
@rultor merge |
@sereshqua/z please review this job completed by @victornoel/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed |
@victornoel please make sure you will find at least 3 issues during next CR, thanks |
@sereshqua yes, thx |
@0crat quality acceptable |
Order was finished, quality is "acceptable": +15 point(s) just awarded to @victornoel/z |
Quality review completed: +4 point(s) just awarded to @sereshqua/z |
#410 - added active users file