Giter Site home page Giter Site logo

Comments (6)

navidshaikh avatar navidshaikh commented on July 21, 2024 1

IMO, we should aim this as first thing after v0.1.

from client.

navidshaikh avatar navidshaikh commented on July 21, 2024 1

This is completed, thanks @maximilien !

from client.

maximilien avatar maximilien commented on July 21, 2024

So I took at pass at this when I was doing the plugin command. It's doable but requires the following changes:

  1. move some types from root.go to a common package to avoid import cycles, e.g., this common package for one of my plugin branches

  2. sweeping change to each command to use the common package above, e.g., this plugins example

Bottom line is that it's not hard to do just requires agreement and one swoop change.

If you guys want this before milestone 1 I could do this this week, while you all asleep in Europe. I'll add this to the agenda tomorrow as a low priority issue.

/cc @sixolet @cppforlife

from client.

rhuss avatar rhuss commented on July 21, 2024

@maximilien Is this something you are already on ? I'm asking while I'm working on #14 I also want to split up and separate the commands better (as I want to use some some package scoped vars which shouldn't 'pollute' whole "commands").

I agree that we need a "common" package which should hold the shared code, e.g. KnParams. I also believe that KnParams should be an interface defined in common and an implementation in the root "commands" package.

As I'm on it as well, I would be happy also to take over this task if you don't have a chance to come to is soonish. (there are some other subtle things like how to register subcommands and also the human-readable printers now reference commands directly).

In any case, we should tackle this issue very soon.

from client.

maximilien avatar maximilien commented on July 21, 2024

OK as per discussion today. Let me submit something today. I am sure you guys will have lots of feedback. More soon.

from client.

maximilien avatar maximilien commented on July 21, 2024

Done in #145

from client.

Related Issues (20)

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.