Skip to content
This repository was archived by the owner on Jan 6, 2025. It is now read-only.

build: import individual rx operators #474

Closed
wants to merge 1 commit into from

Conversation

CaerusKaru
Copy link
Member

@CaerusKaru CaerusKaru commented Oct 31, 2017

@CaerusKaru CaerusKaru force-pushed the rxjs branch 2 times, most recently from 5c72f0c to 63709a9 Compare November 7, 2017 07:13
@CaerusKaru
Copy link
Member Author

Build now passing, rebased with master.

cc @ThomasBurleson

@CaerusKaru CaerusKaru force-pushed the rxjs branch 3 times, most recently from 8f7592a to ea4bf72 Compare November 24, 2017 06:19
@CaerusKaru
Copy link
Member Author

Tests now failing on one edge case in new Firefox Quantum, not related to this PR.

* Removes 'rxjs/operators' import in favor of indivial imports,
  adds lint rule to prevent that type of import
* Maintains parity with angular/components#8160
@ThomasBurleson ThomasBurleson added this to the v2.0.0-beta.13 milestone Nov 29, 2017
@ThomasBurleson ThomasBurleson added the P1 Urgent issue that should be resolved before the next re-lease label Nov 29, 2017
@CaerusKaru CaerusKaru changed the title fix: import individual rx operators build: import individual rx operators Nov 30, 2017
@CaerusKaru
Copy link
Member Author

Closed and merged into #523.

@CaerusKaru CaerusKaru closed this Dec 7, 2017
ThomasBurleson pushed a commit that referenced this pull request Dec 14, 2017
Updates rxjs to latest best practices

* fix: import individual rx operators
* Removes 'rxjs/operators' import in favor of indivial imports,
  adds lint rule to prevent that type of import
* Maintains parity with angular/components#8160

Fixes #519, #508, #533. Closes #474.
@CaerusKaru CaerusKaru deleted the rxjs branch January 5, 2018 07:01
atscott pushed a commit to atscott/flex-layout that referenced this pull request Feb 3, 2018
Updates rxjs to latest best practices

* fix: import individual rx operators
* Removes 'rxjs/operators' import in favor of indivial imports,
  adds lint rule to prevent that type of import
* Maintains parity with angular/components#8160

Fixes angular#519, angular#508, angular#533. Closes angular#474.
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build cla: yes P1 Urgent issue that should be resolved before the next re-lease tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants