Skip to content

add self serve tooling for new primitives - #323

Merged
rezrah merged 5 commits into
mainfrom
rezrah/self-serve-tooling
Apr 27, 2022
Merged

rezrah merged 5 commits into
mainfrom
rezrah/self-serve-tooling

Conversation

@rezrah

@rezrah rezrah commented Apr 25, 2022 •

Copy link
Copy Markdown
Contributor

What's changing?

Refactoring v2 tokens to build and export self-serve primitives from @primer/primitives

🎥 Check out the demo: https://www.loom.com/share/a300e866498844bba551fc4b5c6795ae

Why?

Part of https://github.com/github/primer/issues/924

Primer Brand is expected to generate its own primitives, rather than importing the product tokens from in @primer/primitives.

See epic for more details: https://github.com/github/primer/issues/924

How?

  • Converts previous build functions to usage-agnostic tooling
    • Internally dogfoods the tooling to continue building product-specific tokens
    • Exports the tooling so that Primer Brand can build the same tokens, but also override them where necessary

@changeset-bot

changeset-bot Bot commented Apr 25, 2022 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 8fc70d1

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

This PR includes changesets to release 1 package
Name Type
@primer/primitives 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

@vercel

vercel Bot commented Apr 25, 2022 •

Copy link
Copy Markdown

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/primer/primitives/ct8vQ6Wvf8dePPenMsdPqvXYfyFm
✅ Preview: https://primitives-git-rezrah-self-serve-tooling-primer.vercel.app

@github-actions

github-actions Bot commented Apr 25, 2022 •

Copy link
Copy Markdown
Contributor
Variables changed
No variables changed

@rezrah
rezrah temporarily deployed to github-pages April 25, 2022 13:38 Inactive
@langermank
langermank marked this pull request as ready for review April 27, 2022 15:55
@langermank
langermank requested a review from a team as a code owner April 27, 2022 15:55
@langermank
langermank requested review from a team, pksjce and simurai April 27, 2022 15:55

@langermank langermank 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.

Incredible work Reza! I pulled and built locally, everything looks good to me 👍

@rezrah
rezrah temporarily deployed to github-pages April 27, 2022 16:40 Inactive
@rezrah
rezrah temporarily deployed to github-pages April 27, 2022 16:49 Inactive
@rezrah
rezrah merged commit 40f15d5 into main Apr 27, 2022
@rezrah
rezrah deleted the rezrah/self-serve-tooling branch April 27, 2022 17:15
@primer-css primer-css mentioned this pull request Apr 27, 2022

This branch was previously deployed

1 inactive deployment
github-pages — 8fc70d12 Deployed Apr 27, 2022 by rezrah
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