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

new fixtures for semantic UAST #146

Merged
merged 6 commits into from
Mar 14, 2018
Merged

new fixtures for semantic UAST #146

merged 6 commits into from
Mar 14, 2018

Conversation

juanjux
Copy link
Contributor

@juanjux juanjux commented Mar 1, 2018

I also removed some of the existing ones to avoid duplicates.

@juanjux juanjux self-assigned this Mar 1, 2018
@juanjux juanjux requested a review from dennwc March 1, 2018 10:48
@juanjux
Copy link
Contributor Author

juanjux commented Mar 14, 2018

I'm going to rebase and update the test to the latest spec. We should do the same for all drivers.

Juanjo Alvarez added 6 commits March 14, 2018 11:28
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
Signed-off-by: Juanjo Alvarez <juanjo@sourced.tech>
@juanjux
Copy link
Contributor Author

juanjux commented Mar 14, 2018

Tests updated, PTAL.

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.

New naming rocks! LGTM

@juanjux juanjux merged commit 9ab2824 into bblfsh:master Mar 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants