-
Notifications
You must be signed in to change notification settings - Fork 42
Conversation
feel free to have a look at the cc: @vlerenc and @petersutter |
this - I see you already did it |
the PR looks good for me, except maybe for one thing: I would expect that when choosing a target from the history list, that the chosen target would appear at the top of the list. Not sure what the others think about that. |
sounds reasonable, done in 82f980a |
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.
/lgtm
/merge squash |
/help |
add history write add target name into local history file
What this PR does / why we need it:
add History Targetting the idea comes from #499 (comment) in
History Targetting
sectionWhich issue(s) this PR fixes:
Fixes #511
Special notes for your reviewer:
Release note: