-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix: Added support for Array schema #2
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you not go for npm-shrinkwrap? Just a question out of curiousity.
Are you meaning shrinkwrap vs package-lock? As far as I'm aware package-lock.json is essentially a new version of the shrinkwrap functionality, but is a bit easier to use as it just works with |
btw, are you meaning to re-approve the PR so much? 😛 |
Ha, I was looking at the review functionality in GitHub. :P |
Fair enough! lmk if you want me to change anything |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@edcarroll add yourself to package.json authors and revert the name.
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "normalizr-decorators", | |||
"version": "1.0.0-alpha.1", | |||
"name": "@edcarroll/normalizr-decorators", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove only
package.json
Outdated
"name": "normalizr-decorators", | ||
"version": "1.0.0-alpha.1", | ||
"name": "@edcarroll/normalizr-decorators", | ||
"version": "1.0.0-alpha.5", | ||
"description": "Schema decorators for normalizr", | ||
"author": "Michael Wolbert <michael.wolbert@gmail.com>", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add yourself here please! :)
Add Edward Carroll
Thank you for sorting that 🎉 currently underwater with a work project, but should be wrapped up on Thursday so will be able to take a look at & have a chat about some of the other issues 😄 |
Closes #1
This PR also addresses various other things within the project, like tests & ensuring that the typings are found correctly, as well as removing compiled files from the repo.