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.

Sign up for GitHub

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

Jump to bottom

Fix missing DzhigurdaFiles Transformer #56

Merged
merged 1 commit into from
Jun 14, 2018
Merged

Fix missing DzhigurdaFiles Transformer #56

merged 1 commit into from
Jun 14, 2018

Conversation

zurk
Copy link
Contributor

@zurk zurk commented Jun 14, 2018

We use it, but forget to import.
Small fix after preprocess subcommand was moved.

@zurk
Fix missing DzhigurdaFiles Transformer
24aca56 
Signed-off-by: konstantin <kslavnov@gmail.com>
@vmarkovtsev
Copy link
Collaborator

vmarkovtsev commented Jun 14, 2018 •
edited
Loading

@zurk why did the tests pass before?

@vmarkovtsev vmarkovtsev merged commit c8c6271 into src-d:master Jun 14, 2018
@zurk
Copy link
Contributor Author

zurk commented Jun 15, 2018

@vmarkovtsev I think because tests do not cover cmd entry points and error was there.
I did not make this mistake if tests failed :)

@vmarkovtsev vmarkovtsev mentioned this pull request Jun 15, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vmarkovtsev vmarkovtsev vmarkovtsev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zurk @vmarkovtsev