Skip to content

fix: generate also depends on generate-* targets#665

Merged
talos-bot merged 1 commit into
siderolabs:mainfrom
majabojarska:fix/check-dirty-deps-on-gen-front
Jun 2, 2026
Merged

fix: generate also depends on generate-* targets#665
talos-bot merged 1 commit into
siderolabs:mainfrom
majabojarska:fix/check-dirty-deps-on-gen-front

Conversation

@majabojarska

@majabojarska majabojarska commented Jun 2, 2026

Copy link
Copy Markdown
Member

Resolves #664

  • The generate didn't depend on the additional generate-* targets, which made it incomplete for some repositories (e.g. Omni). Since check-dirty uses the generate target to check for stale artifacts, this lead to stale generate-frontend outputs in Omni, as check-dirty didn't catch it (link).
  • Kres' own generated configs are up to date without any diff, since Kres doesn't have additional generate-* targets.

@majabojarska majabojarska self-assigned this Jun 2, 2026
@github-project-automation github-project-automation Bot moved this to To Do in Planning Jun 2, 2026
@majabojarska majabojarska force-pushed the fix/check-dirty-deps-on-gen-front branch from d7b364a to 5683765 Compare June 2, 2026 10:24
@majabojarska majabojarska marked this pull request as ready for review June 2, 2026 10:24
@talos-bot talos-bot moved this from To Do to In Review in Planning Jun 2, 2026
@majabojarska majabojarska marked this pull request as draft June 2, 2026 10:26
This also fixes a blindspot in the check-dirty target, which would miss
stale artifacts if they were generated by additional "generate-*"
targets.

Signed-off-by: Maja Bojarska <maja.bojarska@siderolabs.com>
@majabojarska majabojarska force-pushed the fix/check-dirty-deps-on-gen-front branch from 5683765 to d3457c9 Compare June 2, 2026 10:30
@majabojarska majabojarska marked this pull request as ready for review June 2, 2026 10:32
@majabojarska majabojarska changed the title fix: check-dirty also depends on generate-* targets fix: generate also depends on generate-* targets Jun 2, 2026
@github-project-automation github-project-automation Bot moved this from In Review to Approved in Planning Jun 2, 2026
@majabojarska

Copy link
Copy Markdown
Member Author

/m

@talos-bot talos-bot merged commit d3457c9 into siderolabs:main Jun 2, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in Planning Jun 2, 2026
@majabojarska majabojarska deleted the fix/check-dirty-deps-on-gen-front branch June 2, 2026 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Make target check-dirty doesn't depend on generate-frontend

4 participants