Skip to content

added git dirty check for our codegen, if the codegen is dirty, build will fail#1230

Merged
ethankonk merged 3 commits intomainfrom
ethan/git-dirty-for-codegen
Mar 3, 2026
Merged

added git dirty check for our codegen, if the codegen is dirty, build will fail#1230
ethankonk merged 3 commits intomainfrom
ethan/git-dirty-for-codegen

Conversation

@ethankonk
Copy link
Contributor

Summary & Motivation

How I Tested These Changes

Did you add a changeset?

If updating one of our packages, you'll likely need to add a changeset to your PR. To do so, run pnpm changeset. pnpm changeset will generate a file where you should write a human friendly message about the changes. Note how this (example) includes the package name (should be auto added by the command) along with the type of semver change (major.minor.patch) (which you should set).

These changes will be used at release time to determine what packages to publish and how to bump their version. For more context see this comment.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Mar 2, 2026

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 3398038:

Sandbox Source
@turnkey/example-react-components Configuration

Copy link
Contributor

@moeodeh3 moeodeh3 left a comment

Choose a reason for hiding this comment

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

this is a good first step, but I’d like to point out that it doesn’t work for @turnkey/http, since the codegen script lives outside of this repo

still awesome sauce 👍

@ethankonk ethankonk merged commit 3eb02cb into main Mar 3, 2026
8 checks passed
@ethankonk ethankonk deleted the ethan/git-dirty-for-codegen branch March 3, 2026 17:28
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