Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

discussion: reduce number of packages in the sdk #119

@LukeWinikates

Description

@LukeWinikates

I think this sdk should have a single public package, and that package should be named wavefront.

I've noticed that telegraf imports only the senders package from this repo, and aliases it to wavefront

import (
    ...
	wavefront "github.com/wavefronthq/wavefront-sdk-go/senders"
)

that means they create a new sender by invoking wavefront.NewSender, which I think looks better than senders.NewSender

I think we could consider moving the exported members from senders, histogram, and event into a wavefront directory/package, and move most of their unexported members into internal.

This would be a big change, but I do think it would make the API of the SDK easier to learn and so I'd like to consider changing this before v1

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions