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

lookout-gometalint-analyzer implementation #1

Merged
merged 28 commits into from
Aug 30, 2018
Merged

lookout-gometalint-analyzer implementation #1

merged 28 commits into from
Aug 30, 2018

Conversation

bzz
Copy link
Contributor

@bzz bzz commented Aug 29, 2018

Initial implementation of test analyzer that wraps gometalint and fixes src-d/lookout#136

  • _fixtures are used in tests to trigger specific comments
  • util/grpchelper is a copy of lookout (due to dependencies on grpc that both projects /vendor/)

the goal is not to have a production-ready analyzer, but rather to gain experience writing analyzers before publishing SDK and to have something operational on staging.

TODOs:

  • build, CI, docs, etc
  • analyzer impl
  • k8s service

bzz added 18 commits August 17, 2018 21:38
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
 - https://github.com/src-d/lookout/tree/v0.1.1/sdk
 - generated code
 - vendoring all dependencies for it

Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
@bzz bzz force-pushed the master branch 2 times, most recently from ee3be06 to 30cdc9f Compare August 29, 2018 08:49
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
@bzz bzz force-pushed the master branch 5 times, most recently from 34f883a to 353160d Compare August 29, 2018 20:59
@bzz
Copy link
Contributor Author

bzz commented Aug 29, 2018

\cc @carlosms for a review

@bzz bzz force-pushed the master branch 2 times, most recently from 61dcf31 to 4d68b85 Compare August 30, 2018 08:27
Copy link
Contributor

@carlosms carlosms left a comment

Choose a reason for hiding this comment

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

Looks good to me

Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
@bzz
Copy link
Contributor Author

bzz commented Aug 30, 2018

history re-written to update k8s descriptor

Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
Signed-off-by: Alexander Bezzubov <bzz@apache.org>
@bzz bzz merged commit cacc48c into empty-master Aug 30, 2018
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

2 participants