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

WIP, Move to sdk/v3 and go-client/v4 #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

bzz
Copy link
Contributor

@bzz bzz commented Jun 4, 2019

Progress


This change is Reviewable

Signed-off-by: Alexander Bezzubov <bzz@apache.org>
@bzz bzz self-assigned this Jun 4, 2019
@bzz bzz requested a review from dennwc June 4, 2019 17:02
Copy link
Member

@dennwc dennwc left a comment

Choose a reason for hiding this comment

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

This part of the change looks good! Not approving yet, since there are few more points in the TODO list.

cmd/bblfsh-tools/common.go Outdated Show resolved Hide resolved
npath.go Outdated Show resolved Hide resolved

// Tokenizer sub-command outputs every token to STDOUT.
Copy link
Member

Choose a reason for hiding this comment

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

I imagine this won't work as one may expect. Tokens are missing for some structures.

bzz and others added 2 commits June 6, 2019 12:05
Co-Authored-By: Denys Smirnov <denis.smirnov.91@gmail.com>
Co-Authored-By: Denys Smirnov <denis.smirnov.91@gmail.com>
@bzz bzz removed their assignment Jun 14, 2019
@bzz
Copy link
Contributor Author

bzz commented Jun 14, 2019

As much as I would love to finish this work that have taught me a lot - I do not think I'll have time for it in next few weeks, so removing myself from assignment for now.

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.

2 participants