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

es6. Closes #124 #125

Merged
merged 1 commit into from
Nov 3, 2015
Merged

es6. Closes #124 #125

merged 1 commit into from
Nov 3, 2015

Conversation

hueniverse
Copy link
Contributor

No description provided.

@hueniverse hueniverse added this to the 6.0.0 milestone Nov 3, 2015
@hueniverse hueniverse assigned hueniverse and unassigned nlf Nov 3, 2015
hueniverse added a commit that referenced this pull request Nov 3, 2015
@hueniverse hueniverse merged commit ed16969 into master Nov 3, 2015
@hueniverse hueniverse deleted the es6 branch November 3, 2015 03:01
stringify: Stringify,
parse: Parse
};
exports.stringify = Stringify;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hueniverse just out of curiosity is there a specific reason for doing this?

@hueniverse
Copy link
Contributor Author

More consistent with other hapi modules.

@AdriVanHoudt
Copy link

Any particular reason they do it? :P
Don't get me wrong I also started with using this style (the object at the bottom) but when the module (or things to export) become large this just makes it more unclear. So I am thinking of moving away from it but was just wondering whether there was a 'real' reason for it other than style/beauty/readability

@hueniverse hueniverse removed their assignment Dec 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants