Skip to content

Add noMutate flag, which addresses #12 and adds the ability to…#15

Merged
maxkorp merged 1 commit into
nodegit:masterfrom
jon-hall:feature/mutating-required-modules-bug
Nov 30, 2015
Merged

Add noMutate flag, which addresses #12 and adds the ability to…#15
maxkorp merged 1 commit into
nodegit:masterfrom
jon-hall:feature/mutating-required-modules-bug

Conversation

@jon-hall

Copy link
Copy Markdown
Contributor

…disable object mutation through the public API.

This ended up needing some fairly significant changes, but I think being able to prevent mutation (particularly for the case of require calls, as in #12) is a very helpful feature that does warrant it.

@maxkorp

maxkorp commented Nov 30, 2015

Copy link
Copy Markdown
Contributor

As i said in your other PR, pending hearing back from other maintainers re: dropping 0.8 support, this should be good to merge 👍.

Thanks a ton for grabbing both of these man ❤️

@maxkorp

maxkorp commented Nov 30, 2015

Copy link
Copy Markdown
Contributor

Can you rebase this on top of master please, to get the updated version matrix (it doesnt appear there were any other changes, so a rebase should be safe).

@jon-hall jon-hall force-pushed the feature/mutating-required-modules-bug branch from 8a5bc4d to 0989ca8 Compare November 30, 2015 16:37
@jon-hall

Copy link
Copy Markdown
Contributor Author

Heh, beat me to it with the other PR! Will rebase and try again 👍.

… disable object mutation through the public API.
@jon-hall jon-hall force-pushed the feature/mutating-required-modules-bug branch from 0989ca8 to 73f8216 Compare November 30, 2015 16:53
@maxkorp

maxkorp commented Nov 30, 2015

Copy link
Copy Markdown
Contributor

💥

maxkorp added a commit that referenced this pull request Nov 30, 2015
…s-bug

Add noMutate flag, which addresses #12 and adds the ability to…
@maxkorp maxkorp merged commit 6551f38 into nodegit:master Nov 30, 2015
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