Skip to content

Conversation

@MikeSpreitzer
Copy link
Contributor

@MikeSpreitzer MikeSpreitzer commented Jan 15, 2026

This PR is here to ask a question: why does compiling an agentic workflow make a change?

me@mymac docs % git status
On branch main
Your branch is up to date with 'upstream/main'.

nothing to commit, working tree clean

me@mymac docs % gh extension install githubnext/gh-aw
✓ Installed extension githubnext/gh-aw

me@mymac docs % gh aw compile scan-merged-prs   
✓ .github/workflows/scan-merged-prs.md (53.7 KB)
⚠ Schedule uses hourly interval with fixed minute offset (0). Consider using fuzzy schedule 'every 1h' instead to distribute workflow execution times and reduce load spikes.
⚠ Compiled 1 workflow(s): 0 error(s), 1 warning(s)

me@mymac docs % git status
On branch main
Your branch is up to date with 'upstream/main'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
	modified:   .gitattributes
	modified:   .github/aw/actions-lock.json
	modified:   .github/workflows/scan-merged-prs.lock.yml

no changes added to commit (use "git add" and/or "git commit -a")

That's all I did. Why did it make these changes?

FYI, following is the typescript of me installing gh-aw just now.

```console
me@mymac docs % gh extension install githubnext/gh-aw
✓ Installed extension githubnext/gh-aw
```

Signed-off-by: Mike Spreitzer <mspreitz@us.ibm.com>
Copilot AI review requested due to automatic review settings January 15, 2026 02:05
@kubestellar-prow kubestellar-prow bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Jan 15, 2026
@kubestellar-prow
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mavrick-1 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@netlify
Copy link

netlify bot commented Jan 15, 2026

Deploy Preview for kubestellar-docs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 6e2e92a
🔍 Latest deploy log https://app.netlify.com/projects/kubestellar-docs/deploys/69684b695eab18000894b9e6
😎 Deploy Preview https://deploy-preview-714--kubestellar-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Contributor

Welcome to KubeStellar! 🚀 Thank you for submitting this Pull Request.

Before your PR can be merged, please ensure:

DCO Sign-off - All commits must be signed off with git commit -s to certify the Developer Certificate of Origin

PR Title - Must start with an emoji: ✨ (feature), 🐛 (bug fix), 📖 (docs), 🌱 (infra/tests), ⚠️ (breaking change)

Getting Started with KubeStellar:

Contributor Resources:


🌟 Help KubeStellar Grow - We Need Adopters!

Our roadmap is driven entirely by adopter feedback. Whether you're using KubeStellar yourself or know someone who could benefit from multi-cluster Kubernetes:

📋 Take our Multi-Cluster Survey - Share your use cases and help shape our direction!


A maintainer will review your PR soon. Feel free to ask questions in the comments or on Slack!

@github-actions github-actions bot added the ci-cd label Jan 15, 2026
@kubestellar-prow kubestellar-prow bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 15, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR demonstrates that running gh aw compile scan-merged-prs modifies several files even on a clean checkout. The changes update action version locks and modify git attributes, raising questions about why compilation produces these modifications.

Changes:

  • Added lock entries for actions/github-script@v8 and githubnext/gh-aw/actions/setup@v0.36.0
  • Removed the .github/workflows/*.campaign.g.md linguist configuration from .gitattributes

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/aw/actions-lock.json Added version locks for two new GitHub Actions
.gitattributes Removed campaign markdown file linguist configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -1,3 +1 @@
.github/workflows/*.lock.yml linguist-generated=true merge=ours
Copy link

Copilot AI Jan 15, 2026

Choose a reason for hiding this comment

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

The removal of the .github/workflows/*.campaign.g.md line appears to be an unintended side effect of the compilation process. If campaign files are still used in the repository, this configuration should be retained to mark them as generated and use the 'ours' merge strategy.

Suggested change
.github/workflows/*.lock.yml linguist-generated=true merge=ours
.github/workflows/*.lock.yml linguist-generated=true merge=ours
.github/workflows/*.campaign.g.md linguist-generated=true merge=ours

Copilot uses AI. Check for mistakes.
@MikeSpreitzer
Copy link
Contributor Author

/hold
This PR is not for merging. It is for asking a question.

@kubestellar-prow kubestellar-prow bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-cd dco-signoff: yes Indicates the PR's author has signed the DCO. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant