Skip to content

[cleanup] Remove unused/duplicated "applyRefactorings" #41

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

rwe
Copy link
Contributor

@rwe rwe commented May 8, 2019

The function was a literally-copied (sans logging) version of Refact.Run.runPipe, but was not used by anything.

The function was a literally-copied (sans logging) version of
`Refact.Run.runPipe`, but was not used by anything.
@mpickering
Copy link
Owner

This function is needed so the program can be used as a library.

@rwe
Copy link
Contributor Author

rwe commented May 9, 2019

Hmm — so the reason I came across it was working on #44 and realizing I was looking at the wrong function. (But then realizing that one of the functions appeared unused). Is there an easy way to unify them? Due to runPipe's interleaved logging, I couldn't see an obvious one.

@rwe
Copy link
Contributor Author

rwe commented May 9, 2019

Ok, closing this and opening #47 which creates a shared implementation.

@rwe rwe closed this May 9, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants