Skip to content

feat: Allow passing custom transforms with instrumentations - #81

Merged
isaacs merged 1 commit into
mainfrom
feat/transforms-with-instrumentations
Oct 4, 2026
Merged

isaacs merged 1 commit into
mainfrom
feat/transforms-with-instrumentations

Conversation

@timfish

@timfish timfish commented Jun 22, 2026 •

Copy link
Copy Markdown
Contributor

The current setup/passing of transforms makes more sense if you're deserialising your instrumentations and therefore can't reference the functions.

This PR adds the ability to pass custom transforms as functions along with the other configuration options.

@isaacs isaacs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see how this would be useful.

@jsumners-nr jsumners-nr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a fan of the added difficulty in reading the configuration, but it's not something that has to be used. So, 🤷‍♂️.

@isaacs
isaacs force-pushed the feat/transforms-with-instrumentations branch from daeb488 to 58f6f90 Compare October 4, 2026 08:40
@isaacs
isaacs merged commit 70ea063 into main Oct 4, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants