Skip to content

docs: add github-app to Allowed Import Fields in imports reference#26119

Merged
pelikhan merged 2 commits intomainfrom
copilot/add-github-app-to-imports
Apr 14, 2026
Merged

docs: add github-app to Allowed Import Fields in imports reference#26119
pelikhan merged 2 commits intomainfrom
copilot/add-github-app-to-imports

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 14, 2026

github-app is a supported top-level import field (compiler merges it with main-workflow-wins precedence) but was absent from the imports reference, leaving users uncertain whether centralizing app credentials in a shared import was intentional or accidentally working.

Changes

  • docs/src/content/docs/reference/imports.md
    • Added github-app: to the Allowed Import Fields list
    • Added github-app: row to the Field-Specific Merge Semantics table with the correct precedence rule: main workflow wins; first imported value fills in if main omits it

Example shared import for the side-repo-ops pattern:

# shared/side-repository.md
---
github-app:
  app-id: ${{ secrets.APP_ID }}
  private-key: ${{ secrets.APP_PRIVATE_KEY }}
---

Any workflow importing this file inherits the credentials without re-declaring them.

Copilot AI changed the title [WIP] Add github-app to allowed import fields in imports reference docs: add github-app to Allowed Import Fields in imports reference Apr 14, 2026
Copilot AI requested a review from pelikhan April 14, 2026 00:49
@pelikhan pelikhan marked this pull request as ready for review April 14, 2026 00:51
Copilot AI review requested due to automatic review settings April 14, 2026 00:51
@pelikhan pelikhan merged commit 6096c38 into main Apr 14, 2026
3 checks passed
@pelikhan pelikhan deleted the copilot/add-github-app-to-imports branch April 14, 2026 00:51
Copy link
Copy Markdown
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

Updates the imports reference documentation to explicitly list github-app: as a supported top-level import field and to document its merge/precedence behavior when provided by shared imports.

Changes:

  • Add github-app: to the “Allowed Import Fields” list for shared workflow imports.
  • Document github-app: merge semantics in the “Field-Specific Merge Semantics” table (main workflow wins; otherwise first imported value is used).
Show a summary per file
File Description
docs/src/content/docs/reference/imports.md Documents github-app: as an allowed import field and clarifies its precedence/merge semantics.

Copilot's findings

Tip

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

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.

Docs: add top-level github-app to Allowed Import Fields in imports reference

3 participants