Skip to content

chore: use checkout to fetch Permify proto files#402

Merged
omer-topal merged 1 commit intomainfrom
chore/use-checkout-for-permify-protos
Mar 26, 2026
Merged

chore: use checkout to fetch Permify proto files#402
omer-topal merged 1 commit intomainfrom
chore/use-checkout-for-permify-protos

Conversation

@omer-topal
Copy link
Copy Markdown
Contributor

@omer-topal omer-topal commented Mar 26, 2026

Summary by CodeRabbit

  • Chores
    • Updated CI workflow to retrieve protocol buffer definitions from an external source and process them locally. Code generation output remains unchanged.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7ae9c497-9c87-444b-9061-66d615d52a04

📥 Commits

Reviewing files that changed from the base of the PR and between adad736 and 1c338ce.

📒 Files selected for processing (1)
  • .github/workflows/protos.yml

📝 Walkthrough

Walkthrough

The workflow configuration for proto generation is modified to source proto files from the Permify/permify repository's proto directory via sparse-checkout, copy them into the local proto/ directory, and proceed with code generation. The Buf CLI setup comments are adjusted to reflect this new sourcing approach.

Changes

Cohort / File(s) Summary
Proto Workflow Configuration
.github/workflows/protos.yml
Added sparse-checkout step to fetch proto files from remote Permify/permify repository into permify-repo/ directory, then copies them to local proto/ directory after cleanup. Updated Buf CLI setup comments to clarify proto sourcing strategy.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 Proto files hop from afar,
Sparse checkout travels like a star,
Copy, generate, build with care—
Permify schemas everywhere!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/use-checkout-for-permify-protos

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@omer-topal omer-topal merged commit 8218595 into main Mar 26, 2026
3 of 4 checks passed
@omer-topal omer-topal deleted the chore/use-checkout-for-permify-protos branch March 26, 2026 15:20
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