Closed martindekov closed 5 years ago
In the YAML files perhaps message
should be messages
?
Could we have an alias of Derek msg
, what do you think? I use GitHub a lot on mobile, less typing is better for me.
Yes Alex your proposals are great I will add those.
Change added and tested:
Response:
When trying to add message whic is not applied, response reads:
Also the config:
/set reviewer: rgee0
Thank you @rgee0 I will refer your comments tomorrow. I am open to suggestions on naming the feature. I went with message
as it was mostly discussed in the Issue.
I think:
Feature toggle and definition within YAML: custom_messages or just messages User command: /message /msg
Does that work? I think it's what Martin landed on
Using message
as the command wasn't mentioned once on #109.
@martindekov if this is meant to be a separate feature then there's further rework required. If I ask how a user would use the message
command without enabling comments
then that should get you thinking in the right area.
You're right. I think this doesn't need to be a toggle, but the feature should be documented with the names as per above.
You are correct again, I suggested "topic" as the command, but Martin chose "message" and I'm ok with that choice.
What I might have seen that somewhere else not sure why I got message in my head I apologise I will swap that for sure.
I will go with topic
I preferred your suggestion of /message and /msg. Please can you keep that as it is?
Thank you @rgee0 for the extended review. I will need to test the change before it is good to go. If you have any more comments let me know. 💯
Also tested e2e on my repo I believe if there are no further commends this is good to go :+1:
Merged 👍 thank you Martin
Adding messages option to the derek bot in order to apply tempaltes or boilerplate things that we write all the time.
Signed-off-by: Martin Dekov (VMware) mdekov@vmware.com
Description
Extending the comments handler, by passing the derek config file (
.DEREK.yml
), in order to take themessages
field from it and pass it to the executing function which sends the message to the Issue.Motivation and Context
Closes #109
How Has This Been Tested?
Manually multi line literal value of the message:![image](https://user-images.githubusercontent.com/34942004/60389645-f9b6f100-9acd-11e9-9b2d-0f31f9eee26a.png)
Whole content of![image](https://user-images.githubusercontent.com/34942004/60389682-f5d79e80-9ace-11e9-869e-d75eaae89d01.png)
.DEREK.yml
:Image is here
martindekov/derek:0.0.7
, in order to test augment your.DEREK.yml
file like pointed in the picture above and swap the images.Types of changes
Checklist:
git commit -s