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

refactor(/internal): remove unused imports #4005

Closed
wants to merge 1 commit into from
Closed

refactor(/internal): remove unused imports #4005

wants to merge 1 commit into from

Conversation

MrOutput
Copy link

fix doc typo
remove unused imports

fix doc typo
remove unused imports
@benlesh
Copy link
Member

benlesh commented Aug 10, 2018

#4006 fixes the travis issue.

@@ -109,7 +109,7 @@ export function forkJoin<T>(...sources: ObservableInput<T>[]): Observable<T[]>;
*
* ### Use forkJoin with project function
* ```javascript
* import { jorkJoin, interval } from 'rxjs';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HAHAHAHAHAHAHAHAHAHAHAAHAHAHA

"jorkJoin"

Copy link
Member

@benlesh benlesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, actually, the unused imports you're removing are actually exported from those files implicitly. (hence some understandable confusion on your part).

You're going to need to readd them.

Perhaps you could just make the exports that use them explicit?

@benlesh
Copy link
Member

benlesh commented Aug 14, 2018

@MrOutput Ping. Do you think you can reduce this PR to just the typo fix? We can't remove the imports, they're implicit exported types, and it breaks the build.

@pertrai1
Copy link
Contributor

@benlesh should this be closed now that #4028 has been merged?

@benlesh benlesh closed this Aug 27, 2018
@MrOutput MrOutput deleted the fix-imports branch August 31, 2018 13:40
@lock lock bot locked as resolved and limited conversation to collaborators Sep 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants