Skip to content
This repository has been archived by the owner on Jan 10, 2018. It is now read-only.

Fix v2 contrib #330

Closed
wants to merge 2 commits into from
Closed

Fix v2 contrib #330

wants to merge 2 commits into from

Conversation

nweldev
Copy link

@nweldev nweldev commented Feb 3, 2017

For now, contributions to master seems to be frozen, waiting for the resolution of #269, which should be released with ngrx/store v3 along with Angular 4 (release planned for march / april 2017).

In the mean time, unit tests doesn't run on the current version (see #321). In order to fix that, and to permit minor features and patch for the v2, this PR "cherry-picked" one of MikeRyan52 commits from #269, and update the CONTRIBUTING documentation.

#319 is deprecated by this PR

MikeRyanDev and others added 2 commits February 3, 2017 13:54
* remove mention of typings add replace npm by yarn in CONTRIBUTING.md
* update yarn.lock

see #269 (especially 30544e9) - fix #321 - close #319
@brandonroberts
Copy link
Member

These changes have been implemented in the monorepo where we are migrating the packages: https://github.com/ngrx/platform

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.

4 participants