-
Notifications
You must be signed in to change notification settings - Fork 156
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
GithubProfile.java:247-250: Implement validation using Kwalify library... #699
Comments
@AlexDag it's yours now, please proceed keeping in mind our principles. Feel free to ask any technical questions right here in the ticket The cost of this task is 30 mins (this is exactly how much will be paid, not less not more), when the task is done |
@AlexDag the ticket is with you for 11 days already. If it's not finished in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle; do not forget that you should remove the |
@alex-palevsky , sorry for my surprised delay,I've some problem with my yahoo mail.So, please, give me 11 days and I'm starting with this task now. Thanks in advance. |
@alex-palevsky we need more time here |
@yegor256 of course, thanks for letting me know, take your time |
@alex-palevsky - we have #731 closed and does not any solutions for usage Kwalify.And Kwalify is a main target for this job.It's a technical question but- Does need to create a new ticket?Or change the specification of the job? |
@alex-palevsky this is postponed. |
@original-brownbear thanks, I added "postponed" label |
@original-brownbear got it, someone else will be assigned soon |
@alex-palevsky this depends on #1032 |
@original-brownbear right, let's wait for #1032 |
@original-brownbear Closing, thanks all |
Puzzle
570-b7bcc1c2
insrc/main/java/com/rultor/profiles/GithubProfile.java:247-250
has to be resolved: Implement validation using Kwalify library in separate class called ValidYaml, move this method to that class and move tests from GitHubProfileValidationTest to ValidYamlTest. Remember about removing PMD suppress below.If you have any technical questions, don't ask me, submit new tickets instead
The text was updated successfully, but these errors were encountered: