Skip to content

Issue 133: Mark JTS as optional dependency in janusgraph-solr#189

Merged
twilmes merged 1 commit intoJanusGraph:0.1from
ngageoint:solr-jts-optional
Apr 10, 2017
Merged

Issue 133: Mark JTS as optional dependency in janusgraph-solr#189
twilmes merged 1 commit intoJanusGraph:0.1from
ngageoint:solr-jts-optional

Conversation

@sjudeng
Copy link
Copy Markdown
Contributor

@sjudeng sjudeng commented Apr 5, 2017

Resolves #133

Signed-off-by: sjudeng <sjudeng@users.noreply.github.com>
@janusgraph-bot janusgraph-bot added the cla: yes This PR is compliant with the CLA label Apr 5, 2017
Copy link
Copy Markdown
Contributor

@twilmes twilmes left a comment

Choose a reason for hiding this comment

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

+1

@twilmes
Copy link
Copy Markdown
Contributor

twilmes commented Apr 10, 2017

@sjudeng just want to double check, I'm good to merge this one, right?

@sjudeng
Copy link
Copy Markdown
Contributor Author

sjudeng commented Apr 10, 2017

I'm good with it but I thought we needed one more review.

@twilmes
Copy link
Copy Markdown
Contributor

twilmes commented Apr 10, 2017

As we have the dev process currently, I think we're good to go: "Two committer approvals are required to merge a pull request". It's not explicitly stated, but I think the intent was that if the submitter is a committer, their vote on the PR counts towards the two approvals. I'll go ahead and merge.

@twilmes twilmes merged commit b24d553 into JanusGraph:0.1 Apr 10, 2017
@sjudeng sjudeng deleted the solr-jts-optional branch April 10, 2017 14:14
bwatson-rti-org pushed a commit to bwatson-rti-org/janusgraph that referenced this pull request Mar 9, 2019
Issue 133: Mark JTS as optional dependency in janusgraph-solr
micpod pushed a commit to micpod/janusgraph that referenced this pull request Nov 5, 2019
Issue 133: Mark JTS as optional dependency in janusgraph-solr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This PR is compliant with the CLA index/solr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants