Fix dup alias "t", keep it for "test" only#1159
Fix dup alias "t", keep it for "test" only#1159ErikSchierboom merged 2 commits intoexercism:mainfrom
Conversation
Command alias "t" was registered by both "troubleshoot" and "test" commands. This PR drops it from "troubleshoot" aliases.
|
Hello. Thanks for opening a PR on Exercism 🙂 We ask that all changes to Exercism are discussed on our Community Forum before being opened on GitHub. To enforce this, we automatically close all PRs that are submitted. That doesn't mean your PR is rejected but that we want the initial discussion about it to happen on our forum where a wide range of key contributors across the Exercism ecosystem can weigh in. You can use this link to copy this into a new topic on the forum. If we decide the PR is appropriate, we'll reopen it and continue with it, so please don't delete your local branch. If you're interested in learning more about this auto-responder, please read this blog post. Note: If this PR has been pre-approved, please link back to this PR on the forum thread and a maintainer or staff member will reopen it. |
|
@muzimuzhi Having slept on it a bit, I think my previous suggestion was wrong. We should keep the existing |
I believe it's already been a regression starting from versions 3.2.0 (2023-08-01), that I'll adjust this PR to drop If assigning single-character aliases for each command is an important convention, compared to |
Right, then I'm fine with keep it as is. |
Command alias "t" was registered by both "troubleshoot" and "test" commands. This PR drops it from "troubleshoot" aliases.