Skip to content

[CHORE] Sync Status field for project #85 (Road to schema v2.0.0 and extensions) - #82

Merged
Eric Godwin (ericgodwin) merged 1 commit into
mainfrom
ericgodwin-sync-project-85-status
Aug 17, 2026
Merged

[CHORE] Sync Status field for project #85 (Road to schema v2.0.0 and extensions)#82
Eric Godwin (ericgodwin) merged 1 commit into
mainfrom
ericgodwin-sync-project-85-status

Conversation

@ericgodwin

Copy link
Copy Markdown
Contributor

Adds the Road to schema v2.0.0 and extensions project (#85) to the set of org projects whose Status field is kept in sync.

No logic changes — the action already handles an arbitrary number of projects, and the existing conflict resolution (last write wins, by field updatedAt) applies unchanged.

Changes

  • action.ymlprojectNumbers default 84,78,5484,78,54,85
  • sync-project-status/README.md — updated the projectNumbers reference entry
  • sync-project-status.yml — updated the header comment listing synced projects

Before merging

  • Confirm the overture-project-manager GitHub App has Projects read & write access to project #85 in the org installation settings — otherwise the run will fail or skip it.
  • Confirm project #85's Status options match the other projects (matching is case-insensitive; unmatched names are logged as warnings and skipped).

Testing

  • workflow_dispatch with dry_run: true lists intended changes involving #85 without errors
  • A live run propagates status between #85 and the other projects

Closes #81

Add the Road to schema v2.0.0 and extensions project (#85) to the set of
org projects whose Status field is kept in sync, and update the action
README and workflow header comment to match.

Refs #81

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Signed-off-by: ericgodwin <eric@overturemaps.org>
@ericgodwin Eric Godwin (ericgodwin) changed the title Sync Status field for project #85 (Road to schema v2.0.0 and extensions) [CHORE] Sync Status field for project #85 (Road to schema v2.0.0 and extensions) Aug 17, 2026
@ericgodwin
Eric Godwin (ericgodwin) marked this pull request as ready for review August 17, 2026 20:36
Copilot AI lite review requested due to automatic review settings August 17, 2026 20:36

Copilot AI 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.

Pull request overview

Adds org ProjectV2 #85 (Road to schema v2.0.0 and extensions) to the existing sync-project-status GitHub Action’s default synced project set, keeping the action behavior unchanged while expanding the default configuration and documentation to reflect the new project.

Changes:

  • Updated .github/actions/sync-project-status/action.yml to include project 85 in the default projectNumbers input.
  • Updated .github/actions/sync-project-status/README.md to document the new default project list including #85.
  • Updated .github/workflows/sync-project-status.yml header comment to enumerate the four synced projects.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/sync-project-status.yml Updates workflow header comment to include project #85 in the synced set.
.github/actions/sync-project-status/README.md Updates documented projectNumbers default to include #85 and its project name.
.github/actions/sync-project-status/action.yml Updates projectNumbers default to 84,78,54,85 so #85 is synced by default.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ericgodwin
Eric Godwin (ericgodwin) merged commit 7649014 into main Aug 17, 2026
10 of 12 checks passed
@ericgodwin
Eric Godwin (ericgodwin) deleted the ericgodwin-sync-project-85-status branch August 17, 2026 20:47
@lowlydba

Copy link
Copy Markdown
Contributor

suggestion: add a push trigger scoped to this workflow file, so a change like this project-number bump runs immediately on merge instead of waiting up to 3 hours for the next 17 */3 * * * cron tick.

on:
  push:
    branches: [main]
    paths:
      - .github/workflows/sync-project-status.yml
  schedule:
    - cron: "17 */3 * * *"
  workflow_dispatch:
    inputs:
      dry_run:
        description: "Log intended changes without applying them"
        required: false
        type: boolean
        default: false

Not requesting changes, this PR's diff doesn't touch the on: block so I can't leave it as an inline suggestion. Worth a follow-up if it'd help catch trigger-breaking edits to this file faster.

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.

Sync Status field for project #85 (Road to schema v2.0.0 and extensions)

3 participants