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

feat: generate tokens with theo #136

Merged
merged 10 commits into from
Jan 5, 2021
Merged

feat: generate tokens with theo #136

merged 10 commits into from
Jan 5, 2021

Conversation

josokinas
Copy link
Contributor

Purpose

In order to (later) generate tokens not only for web, but also for iOS and Android, we need one source of aliases & tokens in a format that can be used to generate multi-platform variants.

Approach

Using Theo to generate (for now) web tokens.

I had an idea of running script each time either aliases.json or tokens.json gets modified, then commit the tokens.scss to Git, however due to issues I gave up, also not sure having tokens.scss in the repo would be much benefitial.

Testing

Tested locally, pipeline should also pass, preview generated.

Risks

N/A

@josokinas josokinas requested a review from rabelloo December 29, 2020 13:52
@josokinas josokinas self-assigned this Dec 29, 2020
@github-actions
Copy link
Contributor

github-actions bot commented Dec 29, 2020

😭 Deploy PR Preview caee119 failed. Build logs

🤖 By surge-preview

@josokinas
Copy link
Contributor Author

The build is failing because workflow runs out of main branch, and this now has adjustment to generate the token. Once merged it will start passing.

CONTRIBUTING.md Outdated Show resolved Hide resolved
scripts/formats/tokens.scss.hbs Outdated Show resolved Hide resolved
scripts/generate.ts Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@josokinas josokinas merged commit 2669588 into main Jan 5, 2021
@josokinas josokinas deleted the feat/theo-tokens branch January 5, 2021 11:07
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