Skip to content

Add targets to keep our GitHub Actions config up-to-date#138

Merged
KushalP merged 1 commit intoapple:mainfrom
KushalP:github-workflows-config-consistency
Dec 2, 2025
Merged

Add targets to keep our GitHub Actions config up-to-date#138
KushalP merged 1 commit intoapple:mainfrom
KushalP:github-workflows-config-consistency

Conversation

@KushalP
Copy link
Copy Markdown
Contributor

@KushalP KushalP commented Dec 2, 2025

Fixes #129.

pkl.project(name=pklproject_github_actions)

The new repository is a dev_dependency. We don't want to ship it as part of rules_pkl because .github won't be visible to users.

@KushalP KushalP enabled auto-merge (squash) December 2, 2025 14:48
@KushalP KushalP force-pushed the github-workflows-config-consistency branch 2 times, most recently from ac13570 to eb581e9 Compare December 2, 2025 14:49
Fixes apple#129.

`pkl.project(name=pklproject_github_actions)`
---------------------------------------------

The new repository is a `dev_dependency`. We don't want to ship it as
part of `rules_pkl` because `.github` won't be visible to users.
@KushalP KushalP force-pushed the github-workflows-config-consistency branch from eb581e9 to 960dc26 Compare December 2, 2025 18:01
@KushalP KushalP disabled auto-merge December 2, 2025 18:02
@KushalP KushalP enabled auto-merge (squash) December 2, 2025 18:02
Copy link
Copy Markdown
Member

@bioball bioball left a comment

Choose a reason for hiding this comment

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

LGTM. How do you run this, btw?

@KushalP KushalP merged commit caf3cf3 into apple:main Dec 2, 2025
8 of 9 checks passed
Comment thread .github/BUILD.bazel
name = "index",
srcs = ["index.pkl"],
multiple_outputs = True,
no_cache = True,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why no cache?

KushalP added a commit to KushalP/rules_pkl that referenced this pull request Dec 2, 2025
KushalP added a commit that referenced this pull request Dec 2, 2025
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.

Let Bazel own (and test) whether the Github Actions workflow yaml files are up-to-date

2 participants