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

Error when using --changes-output-path #20

Closed
oskargargas opened this issue Jul 23, 2021 · 2 comments · Fixed by #22
Closed

Error when using --changes-output-path #20

oskargargas opened this issue Jul 23, 2021 · 2 comments · Fixed by #22

Comments

@oskargargas
Copy link

Hi, I wanted to preview what changes will be made. I used --changes-output-path option. Unfortunately it finished with an error. Regular merge (without --changes-output-path) worked like a charm! This tool is a lifesaver, thanks for it! :D

Traceback (most recent call last):
	6: from /Users/oskar/.rvm/gems/ruby-2.7.1@iko3-ios/bin/ruby_executable_hooks:22:in `<main>'
	5: from /Users/oskar/.rvm/gems/ruby-2.7.1@iko3-ios/bin/ruby_executable_hooks:22:in `eval'
	4: from /Users/oskar/.rvm/gems/ruby-2.7.1@iko3-ios/bin/kintsugi:23:in `<main>'
	3: from /Users/oskar/.rvm/gems/ruby-2.7.1@iko3-ios/bin/kintsugi:23:in `load'
	2: from /Users/oskar/.rvm/gems/ruby-2.7.1@iko3-ios/gems/kintsugi-0.2.0/bin/kintsugi:31:in `<top (required)>'
	1: from /Users/oskar/.rvm/gems/ruby-2.7.1@iko3-ios/gems/kintsugi-0.2.0/lib/kintsugi/cli.rb:94:in `block in create_root_command'
/Users/oskar/.rvm/gems/ruby-2.7.1@iko3-ios/gems/kintsugi-0.2.0/lib/kintsugi.rb:39:in `resolve_conflicts': uninitialized constant #<Class:Kintsugi>::JSON (NameError)```
@byohay
Copy link
Collaborator

byohay commented Jul 30, 2021

Totally right! I probably missed an import during a recent refactoring. Will fix it soon 👍

@byohay byohay linked a pull request Aug 3, 2021 that will close this issue
@byohay byohay closed this as completed in #22 Aug 8, 2021
@byohay
Copy link
Collaborator

byohay commented Aug 8, 2021

@oskargargas the fix was merged, PLMK if there are any problems still

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 a pull request may close this issue.

2 participants