relay icon indicating copy to clipboard operation
relay copied to clipboard

Release v1.0.0

Open aloker opened this issue 6 years ago • 7 comments

I suggest pulling #26 and releasing v0.6.0. We've been successfully using the changes in production for quite some time now.

aloker avatar Sep 10 '18 06:09 aloker

Any news on this ?

tvildo avatar Nov 08 '18 12:11 tvildo

Current 0.5.0 release doesn't appear to be compatible with latest 2.4.0 release of GraphQL.Net, I've had to build locally in order to get it working. Any ETA on when an updated release might be likely?

jim-paton avatar Dec 21 '18 12:12 jim-paton

Hello contributors! I have one question. The package published on Nuget is very old (published in late 2017) and as mentioned it doesn't work with the latest version of GraphQL.NET. I've been using this library by compiling the repo source and not having any issues. Any chance we can have a 0.6.0 release on NuGet?

fiyazbinhasan avatar Apr 24 '21 09:04 fiyazbinhasan

It will be 1.0.0. See https://github.com/graphql-dotnet/relay/milestone/1 . This project is not in the focus of my attention. Of course, it has not been updated for a long time. I just do not have enough time and strength to support all projects that I interested in. @Shane32 may help. Also we need to configure CI pipeline and some other things like code style.

sungam3r avatar Apr 26 '21 06:04 sungam3r

I wouldn't mind releasing it now as 0.6.0, but I don't think the current state of the code is worthy of a 1.0.0 release.

It needs:

  • All example and sample projects to run properly (they do not do so now)
  • Sample projects' dependencies to be brought up to date
  • Basic tests that exercise each sample project
  • Basic tests for the core functionality provided
  • APIApprovals tests added
  • GitHub Action workflows for tests and publishing
  • Add code style rules and formatting tests
  • Review of included public members to see if they are appropriate and do not mirror existing functionality (e.g. MutationInputs is identical to Inputs)
  • Review of if/how this project would be useful in a typical scenario (with a backing database/repository) versus only for in-memory databases
  • Basic review of how these classes can interact with the other portions of the published libraries, such as dataloaders or authentication.
  • Review if connection builders are provided and/or how they interact or conflict with the connection builders provided with the main GraphQL.NET library.
  • Ensure that the existing documentation is accurate
  • Ensure that at a minimum, some very basic documentation exists
  • Preferably, add xml comments to all public members

These are all tasks that should be completed regardless of the feature set provided. None of it is very hard, but it does take time.

I believe that releasing the current code as a 0.6.0 release would allow other developers to test and provide more feedback and/or help to this project.

If this sounds good, I can add in some GitHub Action scripts so it will publish, and we can release 0.6.0 once #88 is merged. I don't have the time to put in the necessary to work on this to bring it to a 1.0.0, and it sounds like @sungam3r doesn't either. So unless someone else volunteers to work on everything that needs to be done, it may be beneficial for the community to release 0.6.0 in the meantime. The currently published 0.5.0 release is useless to anyone, as it depends on GraphQL.NET 0.14.0.633.

Shane32 avatar Apr 28 '21 04:04 Shane32

Agree. One thing I ask you is to post all the mentioned above tasks as issues.

sungam3r avatar Apr 28 '21 06:04 sungam3r

@fiyazbinhasan I'll release 0.6.0 once #104 and #88 are merged.

Shane32 avatar Apr 28 '21 16:04 Shane32