Skip to content

chore: change repository owner for declarative setup#3048

Merged
levivannoort merged 1 commit into
mainfrom
change-repository-owner
Jun 11, 2026
Merged

chore: change repository owner for declarative setup#3048
levivannoort merged 1 commit into
mainfrom
change-repository-owner

Conversation

@levivannoort

Copy link
Copy Markdown
Member

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@appwrite

appwrite Bot commented Jun 11, 2026

Copy link
Copy Markdown

Appwrite Website

Project ID: 69d7efb00023389e8d27

Sites (1)
Site Status Logs Preview QR
 website
69d7f2670014e24571ca
Ready Ready View Logs Preview URL QR Code

Website (appwrite/website)

Project ID: 684969cb000a2f6c0a02

Sites (1)
Site Status Logs Preview QR
 website
68496a17000f03d62013
Processing Processing View Logs Preview URL QR Code


Tip

Teams feature lets you group users with membership management and role permissions

@greptile-apps

greptile-apps Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes the staging deployment workflow so that the declarative configuration repository (assets-applications) is accessed under the correct GitHub organization (appwrite-labs) instead of the current repository's owner (appwrite).

  • Introduces a new DECLARATIVE_OWNER: appwrite-labs environment variable and replaces both ${{ github.repository_owner }} references in the deploy job — one for generating the GitHub App installation token and one for the actions/checkout repository path — with ${{ env.DECLARATIVE_OWNER }}.
  • The change is minimal and self-contained; no other jobs or steps are affected.

Confidence Score: 5/5

Safe to merge — the change is a two-line targeted fix that redirects cross-org repository access to the correct owner.

The diff replaces an implicit runtime value (github.repository_owner, which evaluates to appwrite) with an explicit, correctly scoped constant (appwrite-labs) in exactly the two places that determine where the GitHub App token is installed and where the declarative repo is checked out. Both replacements are consistent, the new variable is clearly named and immediately visible in the env block, and no other workflow logic is touched.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/staging.yml Adds DECLARATIVE_OWNER env var set to "appwrite-labs" and replaces both uses of github.repository_owner in the deploy job with this new variable, correctly targeting the cross-org declarative repository.

Reviews (1): Last reviewed commit: "chore: change repository owner for decla..." | Re-trigger Greptile

@levivannoort levivannoort merged commit 4d5331a into main Jun 11, 2026
13 of 17 checks passed
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.

2 participants