Skip to content

ci: verify builds with the shared workflow - #11

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 event and permission configuration are valid, with no issues found in the changed workflow.

Adds a GitHub Actions caller for the shared verification workflow on pull requests and pushes to main. The caller grants only contents: read and pins the reusable workflow to a commit SHA.

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

Reviewed 12005f2 · 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 a2032ca into main Nov 3, 2026
4 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the ci/verify branch November 3, 2026 20:05
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