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.

Sign up for GitHub

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

Jump to bottom

InputWritr: a first revamp #319

Merged

JoshuaKGoldberg merged 2 commits into main from inputwritr-first-rewrite

Aug 21, 2022

Member

JoshuaKGoldberg commented Aug 21, 2022 •

edited

Loading

Cleans up InputWritr a bit by improving some of the docs and function names.

Adds partial unit test coverage
Moves alias conversion to UserWrappr, where it belongs
Renames makePipe to createPipe (I like "create" more for this)_
Removes a few unused areas of code

Filed #317 to tackle a more complete overhaul of the package long term.

JoshuaKGoldberg added 2 commits

August 21, 2022 10:45


          InputWritr: a first revamp

68943ba


          Complete makePipe->createPipe rename

6f5435e

JoshuaKGoldberg marked this pull request as ready for review

August 21, 2022 15:00

JoshuaKGoldberg merged commit e6dc76e into main

JoshuaKGoldberg deleted the inputwritr-first-rewrite branch

August 21, 2022 15:00

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment