Skip to content

ci: verify builds with the shared workflow - #8

Merged
ChiragAgg5k merged 1 commit into
mainfrom
ci/verify
Nov 3, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
ci/verify

Conversation

@ChiragAgg5k

@ChiragAgg5k ChiragAgg5k commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

This repository has no CI of its own, so Renovate PRs here are skipped by the org auto-merge rule as "repository has no CI".

This adds a caller for the shared verify workflow added in appwrite/renovate-config#12. It detects each project in the repository, installs it from the lockfile, and runs its build and test scripts when they exist. It runs with a read-only token and no secrets.

The reusable workflow is pinned by commit SHA, and Renovate keeps the pin up to date.

@hansi-codes

hansi-codes Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The reusable workflow reference and call configuration are valid, and the caller grants only read access.

Adds a GitHub Actions workflow that invokes Appwrite’s pinned shared verification workflow for pull requests and pushes to main. The caller grants read-only repository contents access and passes no secrets.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 1
File Change
.github/workflows/verify.yml Calls the shared verify workflow for pull requests and main-branch pushes with read-only permissions.

Reviewed dd91e31 · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 17c2e79 into main Nov 3, 2026
3 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the ci/verify branch November 3, 2026 20:06
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.

1 participant