Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide transform options type #323

Merged

Conversation

DarioSoller
Copy link
Contributor

@DarioSoller DarioSoller commented Oct 28, 2024

As discussed in issue #319 , this PR exposes the TransformOptions type. Further it fixes type definition file override errors, when running the build locally.

I packed the sd-transform package locally via npm pack and tested the import of the TransformOption type like this:

import { TransformOptions } from '@tokens-studio/sd-transforms';

and it worked like expected!

Copy link

changeset-bot bot commented Oct 28, 2024

🦋 Changeset detected

Latest commit: 3a841d2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@tokens-studio/sd-transforms Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@DarioSoller
Copy link
Contributor Author

I have just seen the changeset bot message. What version bump would you do for this change, if any? I can adjust the commit messages accordingly as soon as I know.

src/index.ts Outdated Show resolved Hide resolved
@jorenbroekema
Copy link
Member

A patch bump works for me!

@DarioSoller DarioSoller force-pushed the provide-transform-options-type branch from c137579 to 8458973 Compare October 28, 2024 21:28
@jorenbroekema
Copy link
Member

Could you run npx changeset locally for creating the changeset entry? Make sure to commit the file as well

@jorenbroekema jorenbroekema merged commit fb3c7d3 into tokens-studio:main Nov 5, 2024
2 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.

2 participants