[ 
https://issues.apache.org/jira/browse/TINKERPOP-1552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16044488#comment-16044488
 ] 

ASF GitHub Bot commented on TINKERPOP-1552:
-------------------------------------------

GitHub user FlorianHockmann opened a pull request:

    https://github.com/apache/tinkerpop/pull/623

    TINKERPOP-1552: Minor code cleanup for Gremlin.Net

    https://issues.apache.org/jira/browse/TINKERPOP-1552
    
    This mainly removes explicit type castings that are now obsolete thanks to 
changes from the pull request #620 from @jorgebay. It also makes the Dictionary 
CSharpToJavaEnums in NamingConversions private to avoid direct access to it as 
GetEnumJavaName should be used instead.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/FlorianHockmann/tinkerpop dotnet-glv-cleanup

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tinkerpop/pull/623.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 #623
    
----
commit 68c5a0cbb368f18f1e8650a84eee2f56e3bc2507
Author: Florian Hockmann <f...@florian-hockmann.de>
Date:   2017-06-09T14:14:49Z

    Minor code cleanup for Gremlin.Net
    
    This mainly removes explicit type casting that are now obsolete thanks to 
changes from the pull request #620. It also makes the Dictionary 
CSharpToJavaEnums in NamingConversions private to avoid direct access to it as 
GetEnumJavaName should be used instead.

----


> C# Gremlin Language Variant
> ---------------------------
>
>                 Key: TINKERPOP-1552
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1552
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: language-variant
>    Affects Versions: 3.2.3
>            Reporter: Jorge Bay
>            Assignee: stephen mallette
>
> It would be nice to have a C# GLV that runs under .NET Framework 4.5+ and 
> .NET Core.
> The maven build could use the Exec Maven Plugin to exec .NET Core's [dotnet 
> test|https://www.microsoft.com/net/core#macos] command.
> Some requirements, from the mailing list (edited):
> {quote}
> 1. The GLV should keep in line with class/method names of the java API
> where possible to ensure consistency of feel across languages.
> 2. There needs to be adequate tests (we're still discussing the approach to
> testing GLVs and i think that needs to be tackled sooner than later as more
> GLVs start to come in). Those tests should produce xunit style output
> unless there is some good reason not to.
> 3. There needs to be adequate documentation (e.g. Reference docs)
> 4. The build/deploy process needs to be bound to maven which might be one of 
> the trickier bits to deal with.
> {quote}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to