Skip to content

refactor: have only one descriptors object #640

@alexander-fenster

Description

@alexander-fenster

Right now, the generated code has a non-trivial descriptor object such as

class Client {
  ...
  descriptors: {
    page: { ... },
    stream: { ... },
    longrunning: { ... },
    batching: { ... },
  }
  ...
}

This is actually an overcomplicated object since a method can only be paging, or streaming, or longrunning, or support batching, but not two of those at the same time. We need to replace it with just one descriptors object (key: method name, value: any descriptor).

This will formally be a breaking change because someone could depend on a given descriptor present (we actually gave some advice about that long time ago), so this should wait till the next major version bump.

Metadata

Metadata

Assignees

No one assigned

    Labels

    type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions