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

Add example with input variables #37

Closed
wants to merge 1 commit into from
Closed

Add example with input variables #37

wants to merge 1 commit into from

Conversation

idkjs
Copy link
Contributor

@idkjs idkjs commented Sep 3, 2020

No description provided.

@idkjs idkjs mentioned this pull request Sep 3, 2020
@jeddeloh
Copy link
Owner

jeddeloh commented Sep 3, 2020

Thanks for the contribution! I haven't been able to look at this yet, but a couple things before I do:

  1. Could you look at the package.json changes and confirm everything there is intentional?
  2. Could you remove the yarn.lock files, I thought I'd added that to .gitignore, but I guess not
  3. I would have expected the current mutation examples to include variables. Do they really not? If they do use variables, what are these examples showing that they do not?

Thanks, again! I'll try and get to this soon.

@jfrolich
Copy link
Collaborator

jfrolich commented Sep 4, 2020

It looks like you a git reconciliation issue. You probably started this PR on a very old version of the the library and something went wrong while trying to merge the new changes.

I suggest to always look at the diff before you commit ;)

@idkjs idkjs closed this Sep 4, 2020
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.

None yet

3 participants