Skip to content
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

Remove unused code #751

Merged
merged 1 commit into from
Jan 21, 2019
Merged

Conversation

3mard
Copy link
Contributor

@3mard 3mard commented Dec 25, 2018

Get rid of bunch of unused imports/functions in the index files

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 85.205% when pulling 740f27e on 3mard:refactor/remove-unused-code into d58488b on TypeStrong:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 85.205% when pulling 740f27e on 3mard:refactor/remove-unused-code into d58488b on TypeStrong:master.

@aleclarson
Copy link

@blakeembrey Can this be merged soon, please?

@blakeembrey
Copy link
Member

@aleclarson Is there some sort of urgency here? The prior feature using this code has yet to be released and we need to fix the test suite before landing these changes.

@aleclarson
Copy link

@blakeembrey No, just wondering why it hasn't been merged. :) I fixed the test suite in #757. 👍

@blakeembrey blakeembrey merged commit abcb13b into TypeStrong:master Jan 21, 2019
@3mard 3mard deleted the refactor/remove-unused-code branch January 22, 2019 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants