Skip to content

Skip generation if spec is unchanged#1514

Open
CodeSimcoe wants to merge 1 commit intoquarkiverse:mainfrom
CodeSimcoe:skip-if-unchanged
Open

Skip generation if spec is unchanged#1514
CodeSimcoe wants to merge 1 commit intoquarkiverse:mainfrom
CodeSimcoe:skip-if-unchanged

Conversation

@CodeSimcoe
Copy link
Copy Markdown
Contributor

@CodeSimcoe CodeSimcoe commented Mar 31, 2026

Proposition to address #878

Skip generation if :

  • property is enabled (maybe it could be enabled by default ? I've put it to false for now, for compatibility reasons)
  • spec hasn't changed
  • config hasn't changed

@CodeSimcoe CodeSimcoe requested a review from a team as a code owner March 31, 2026 09:32
Copy link
Copy Markdown
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Looks good! Many thanks for taking your time to help us :)

Copy link
Copy Markdown
Member

@mcruzdev mcruzdev left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this pull request @CodeSimcoe 👍🏼

I left a few comments, see if makes sense 😃

@CodeSimcoe
Copy link
Copy Markdown
Contributor Author

I've made some changes, should be much simpler in terms of configuration management

@mcruzdev mcruzdev requested a review from ricardozanini April 7, 2026 01:47
@mcruzdev
Copy link
Copy Markdown
Member

mcruzdev commented Apr 7, 2026

Hey @CodeSimcoe thanks for the new updates, could you squash and to solve the conflicts?

@CodeSimcoe CodeSimcoe force-pushed the skip-if-unchanged branch 2 times, most recently from 532daef to 8e4eb98 Compare April 7, 2026 13:46
@CodeSimcoe
Copy link
Copy Markdown
Contributor Author

Hey @CodeSimcoe thanks for the new updates, could you squash and to solve the conflicts?

Should be good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants