Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the default Git branch reference used by the /update webhook endpoint to target main by default, and aligns README examples accordingly.
Changes:
- Change the default
branchRefvalue in the webhook handler fromrefs/heads/views-stagingtorefs/heads/main. - Update README
/updateexample payloads to referencerefs/heads/main. - Update README environment variable docs to state a
maindefault.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| src/app.py | Updates the default branchRef used to decide whether to process webhook push events. |
| README.md | Updates /update payload examples and the documented default branch ref. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| * `TEMP`: path to an existing folder to create temporary locks | ||
| * `dataViewsFolder`: path to where your repository of `addon-datastore` is checked out locally | ||
| * `branchRef`: Git branch to track for `addon-datastore` | ||
| * views-staging/views | ||
| * Default is `main` |
There was a problem hiding this comment.
The docs say the branchRef default is main, but the application default is refs/heads/main (and /update compares against the full ref from GitHub payloads). Please update the docs to reflect the full default value and clarify whether branchRef should be a full ref (refs/heads/<branch>) or a short name—also align the earlier flask run example (branchRef=views) accordingly.
No description provided.