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: restore @angular/cdk clone #35

Merged
merged 4 commits into from
Aug 7, 2020

Conversation

nitayneeman
Copy link
Owner

@nitayneeman nitayneeman commented Aug 7, 2020

This PR restores the source files because of #17 (it was removed as part of #33).

@mraible
Copy link
Contributor

mraible commented Aug 7, 2020

@nitayneeman I tried switching to this branch. If you look at my PR, you can see that its Angular tests are now failing.

@nitayneeman
Copy link
Owner Author

@mraible I've managed to reproduce your issue locally (I cloned your project and linked my local branch). I've pushed a couple of changes that align to the sources of @angular/cdk@8.2.3 - which I think was the mismatch that causing the error.

Would you like to give another try?

@mraible
Copy link
Contributor

mraible commented Aug 7, 2020

@nitayneeman I ran all my tests again after your updates and they pass now. Nice work!

@nitayneeman nitayneeman merged commit 3a0bdc0 into master Aug 7, 2020
@nitayneeman nitayneeman deleted the refactor/restore-angular-cdk-clone branch August 7, 2020 18: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.

2 participants