Giter VIP home page Giter VIP logo

Comments (5)

StephenJosey avatar StephenJosey commented on July 19, 2024

Hey @mvastola thanks for the feedback. Just FYI our SDKs are generated automatically and therefore not idiomatic, which is why you're seeing this. I've shared this with the team, though.

from square-ruby-sdk.

mvastola avatar mvastola commented on July 19, 2024

@StephenJosey, ok, but even if a script is used, that script can be changed to make much less repetitive code. As it is now, it's much harder to alter and add features.

Even more importantly though, this implies it's impossible to make PRs against this repo that could be merged. The code used to generate this library also needs to be open sourced so that this becomes possible.

from square-ruby-sdk.

StephenJosey avatar StephenJosey commented on July 19, 2024

We definitely appreciate the feedback, and we're looking into how we can clean this up. However, just for clarification, we will not be open-sourcing the code generator.

from square-ruby-sdk.

havenwood avatar havenwood commented on July 19, 2024

@mvastola PRs are welcome, but it's best to open an issue ticket first to discuss since we'll need to update the generator to match merged PR. As the generated code becomes more idiomatic and less repetitive, PRs should be easier to make and merge. Thank you for the feedback!

from square-ruby-sdk.

mvastola avatar mvastola commented on July 19, 2024

We definitely appreciate the feedback, and we're looking into how we can clean this up. However, just for clarification, we will not be open-sourcing the code generator.

@StephenJosey, this feels a bit weird to me. This brings into question why this repo is even on GitHub if it's not truly open source. I mean I guess there can be bugs, but by posting things on GitHub, companies send the message they embrace open source.

This isn't really a fair deal though -- people (Square's customers) can use this repo to improve your product (because you're willing to take their changes and apply them to your generator), but they then don't benefit from being able to reuse the codebase they're ultimately contributing to. Even worse, this is being done somewhat covertly -- you do not make the fact that this code is machine-generated public knowledge, so people don't know the terms of engagement when they contribute.

It really prompts the question -- why even have this on GitHub if this SDK isn't a truly collaborative effort? Why not just publish direct to rubygems? This isn't really how open source is supposed to work..

from square-ruby-sdk.

Related Issues (16)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.