You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jena.apache.org by ajs6f <gi...@git.apache.org> on 2017/11/29 17:55:26 UTC
[GitHub] jena pull request #321: Just doing some syntax cleanup by using the new Grap...
GitHub user ajs6f opened a pull request:
https://github.com/apache/jena/pull/321
Just doing some syntax cleanup by using the new Graph::find method
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ajs6f/jena FindANYtoFind
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/jena/pull/321.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #321
----
----
---
Re: Fwd: [GitHub] jena pull request #321: Just doing some syntax
cleanup by using the new Grap...
Posted by Andy Seaborne <an...@apache.org>.
Go ahead. The build isn't until next week (as/when I can find the time)
and there are whole bunch of PRs.
For things like this, either it gets in ... or it doesn't. To the
users, it does not make much of a difference.
We have tests ...
Andy
On 29/11/17 17:56, ajs6f wrote:
> Andy-- this is so trivial I would have just committed it, but I didn't want to bump into anything you might be doing to prep for a release.
>
> ajs6f
>
>> Begin forwarded message:
>>
>> From: ajs6f <gi...@git.apache.org>
>> Subject: [GitHub] jena pull request #321: Just doing some syntax cleanup by using the new Grap...
>> Date: November 29, 2017 at 12:55:26 PM EST
>> To: dev@jena.apache.org
>> Reply-To: dev@jena.apache.org
>> Reply-To: dev@jena.apache.org
>>
>> GitHub user ajs6f opened a pull request:
>>
>> https://github.com/apache/jena/pull/321
>>
>> Just doing some syntax cleanup by using the new Graph::find method
>>
>>
>>
>> You can merge this pull request into a Git repository by running:
>>
>> $ git pull https://github.com/ajs6f/jena FindANYtoFind
>>
>> Alternatively you can review and apply these changes as the patch at:
>>
>> https://github.com/apache/jena/pull/321.patch
>>
>> To close this pull request, make a commit to your master/trunk branch
>> with (at least) the following in the commit message:
>>
>> This closes #321
>>
>> ----
>>
>> ----
>>
>>
>> ---
>
>
Fwd: [GitHub] jena pull request #321: Just doing some syntax cleanup
by using the new Grap...
Posted by ajs6f <aj...@apache.org>.
Andy-- this is so trivial I would have just committed it, but I didn't want to bump into anything you might be doing to prep for a release.
ajs6f
> Begin forwarded message:
>
> From: ajs6f <gi...@git.apache.org>
> Subject: [GitHub] jena pull request #321: Just doing some syntax cleanup by using the new Grap...
> Date: November 29, 2017 at 12:55:26 PM EST
> To: dev@jena.apache.org
> Reply-To: dev@jena.apache.org
> Reply-To: dev@jena.apache.org
>
> GitHub user ajs6f opened a pull request:
>
> https://github.com/apache/jena/pull/321
>
> Just doing some syntax cleanup by using the new Graph::find method
>
>
>
> You can merge this pull request into a Git repository by running:
>
> $ git pull https://github.com/ajs6f/jena FindANYtoFind
>
> Alternatively you can review and apply these changes as the patch at:
>
> https://github.com/apache/jena/pull/321.patch
>
> To close this pull request, make a commit to your master/trunk branch
> with (at least) the following in the commit message:
>
> This closes #321
>
> ----
>
> ----
>
>
> ---
[GitHub] jena pull request #321: Just doing some syntax cleanup by using the new Grap...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/jena/pull/321
---