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

Version 2 is here! #78

Open
wants to merge 34 commits into
base: master
Choose a base branch
from
Open

Version 2 is here! #78

wants to merge 34 commits into from

Conversation

weichx
Copy link
Owner

@weichx weichx commented Feb 5, 2018

No description provided.

@TheBinaryFox
Copy link

Is there any ETA on when this version will be published to npm?

@weichx
Copy link
Owner Author

weichx commented Feb 27, 2018

I can publish it as a beta, I have a few more things I want to look into before officially releasing it but haven't had time with chaos at work recently.

@weichx
Copy link
Owner Author

weichx commented Feb 27, 2018

Published, you can pull it with npm install cerialize@next

@weichx weichx closed this Feb 27, 2018
@weichx weichx reopened this Feb 27, 2018
@TheBinaryFox
Copy link

TheBinaryFox commented Feb 28, 2018

Thanks! If it's not too much trouble, could you also build it for commonjs modules?

I end up with this error when using it in Node 9.5.0:

node_modules/cerialize/dist/index.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,global,jest){import { MetaData } from "./meta_data";
                                                                                             ^^^^^^

SyntaxError: Unexpected token import

@joey-he8x
Copy link

run into same error like TheBinaryFox

@weichx
Copy link
Owner Author

weichx commented Mar 2, 2018

@TheBinaryFox @joey-he8x changed the module type to commonjs, hope it fixes your issues

@marcelh-gh
Copy link

@weichx cerialize looks great! :) are you still going to work on v2?

@weichx
Copy link
Owner Author

weichx commented Jul 10, 2018

To be honest not really, as is it should be pretty well tested and functional but I don't work on the web anymore and can't dedicate time to the project right now.

@marcelh-gh
Copy link

@weichx Thanks for the reply :-) if it's ok with you I could have a look at the PR and then finally merge it...

@marcelh-gh
Copy link

@weichx I have reviewed about a dozen of similar libs for serialization in detail recently and even the most popular one (class-transformer) is not really maintained anymore but has lots of open issues and PRs.
I do like the simple and straight-forward approach of cerialize. Since I want to use this in a large production app, I'd rather try to push forward development on this one than bet on class-transformer or serializr.

@weichx
Copy link
Owner Author

weichx commented Jul 11, 2018

@marcelh-gh sure, happy to have another set of eyes on it

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.

5 participants