-
-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: include base in rerun commands #1301
Conversation
LGTM! :D |
@all-contributors please add @niklas-wortmann for code.
|
I've put up a pull request to add @niklas-wortmann! 🎉 I couldn't determine any contributions to add, did you specify any contributions? |
🎉 This is included in version v1.57.3 🎉 The release is available on: Cheers! 📦🚀 |
Adds @niklas-wortmann as a contributor for code. This was requested by JoshuaKGoldberg [in this comment](#1301 (comment)) --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
PR Checklist
status: accepting prs
Overview
Attaches
base
the options returned during augmentation, so they can get printed out later.I noticed that the
exclusionKeys
stuff was exported from theaugmentOptionsWithExcludes.ts
file but never used in its tests. So I extracted it to a newexclusionKeys.ts
.Adds some unit tests, but doesn't do all of #699. There's still coverage missing.
Co-authored-by: @niklas-wortmann