#1039 Use merged and validated of version .rultor.yml for merge command.
#1070
Conversation
|
@xupyprmv while I just give this a quick read to be safe :) |
| new TalkIssues(Profiles.github(), xml).get().repo() | ||
| ); | ||
| profile = this.merged( | ||
| master, |
There was a problem hiding this comment.
@xupyprmv the variable master is redundant here, please inline this real quick :)
|
@xupyprmv found one small thing, not sure if this was in the other PR and I just missed it (can't see it anymore now :/) or somehow new. Either way please fix it :) Thanks! |
|
@original-brownbear I modified PR description and fixed code. Great thanks for help. Please continue. |
|
@rultor merge please :) |
|
@xupyprmv thanks for the good job :) |
@original-brownbear OK, I'll try to merge now. You can check the progress of the merge here |
@original-brownbear Done! FYI, the full log is here (took me 17min) |
|
@rultor deploy |
@original-brownbear OK, I'll try to deploy now. You can check the progress here |
@original-brownbear Done! FYI, the full log is here (took me 7min) |
#1039 I implemented validated method, that checks that merged .rultor.yml has no changes in architect and commander sections. I covered
validatedmethod by unit tests. I added one puzzle (to implementmergedmethod, write IT forvalidatedmethod and use it.This is a copy of #1064 that cannot be reopened. That's why I created new one.