Skip to content
This repository has been archived by the owner on Mar 8, 2020. It is now read-only.

Do the duplicate removal on all node roles, not only the new ones. #202

Merged
merged 2 commits into from
Nov 7, 2017
Merged

Do the duplicate removal on all node roles, not only the new ones. #202

merged 2 commits into from
Nov 7, 2017

Conversation

juanjux
Copy link
Contributor

@juanjux juanjux commented Nov 7, 2017

Fixes bblfsh/python-driver/issues/130

Signed-off-by: Juanjo Alvarez juanjo@sourced.tech

Juanjo Alvarez added 2 commits November 7, 2017 14:50
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
@juanjux juanjux self-assigned this Nov 7, 2017
@juanjux juanjux requested a review from abeaumont November 7, 2017 14:01
@codecov
Copy link

codecov bot commented Nov 7, 2017

Codecov Report

Merging #202 into master will increase coverage by 0.06%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #202      +/-   ##
==========================================
+ Coverage   32.12%   32.19%   +0.06%     
==========================================
  Files          23       23              
  Lines        2957     2960       +3     
==========================================
+ Hits          950      953       +3     
  Misses       1901     1901              
  Partials      106      106
Impacted Files Coverage Δ
uast/ann/ann.go 98.97% <100%> (+0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02f578d...d424c2d. Read the comment docs.

@juanjux juanjux merged commit 92df1df into bblfsh:master Nov 7, 2017
@juanjux juanjux deleted the fix/unique_roles branch November 13, 2017 11:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants