Wanted to PR in a feature to remove the SSH config changes kr makes during the pair command. Motivation is to support shared machines with kr installed whose users are not all entirely using kr(i.e. when pair-programming with someone who does not want to use kr)
Given that the pair command always attempts to modify SSH config as kr desires, I figured it should be fine to remove the config on unpair command. This proposed change prevents modification to SSH config if opting out via the KR_SKIP_SSH_CONFIG env var.
Test I modified to drive out development isn't the cleanest I understand; however, I didn't want to make big refactoring decisions for those tests without some input from y'all first on whether or not this feature is something you'd be fine introducing. Would still of course love to hear any feedback and of course if there is any issue with this!
Hello Krypton folks,
Wanted to PR in a feature to remove the SSH config changes
kr
makes during thepair
command. Motivation is to support shared machines withkr
installed whose users are not all entirely usingkr
(i.e. when pair-programming with someone who does not want to usekr
)Given that the
pair
command always attempts to modify SSH config askr
desires, I figured it should be fine to remove the config onunpair
command. This proposed change prevents modification to SSH config if opting out via theKR_SKIP_SSH_CONFIG
env var.Test I modified to drive out development isn't the cleanest I understand; however, I didn't want to make big refactoring decisions for those tests without some input from y'all first on whether or not this feature is something you'd be fine introducing. Would still of course love to hear any feedback and of course if there is any issue with this!
Thanks, Jwal