Skip to content

fix: exclude activity-state.json from sync-leaderboard workflow commits #532

Description

@jagdish-15

PR #398 introduced data/activity-state.json, which is written by scripts/analyze-inactivity.js.

The analyze-inactivity.yml workflow correctly commits only inactive-users.json. However, the sync-leaderboard.yml workflow stages changes using git add . and only resets users.json and inactive-users.json, meaning activity-state.json will also be staged and committed on every leaderboard sync and cause conflicts when both workflows run together.

Affected File

  • .github/workflows/sync-leaderboard.yml

Expected Behaviour

activity-state.json should only be committed by analyze-inactivity.yml. The leaderboard sync workflow should ignore it.

Proposed Fix

After the existing git reset commands in sync-leaderboard.yml, add:

git reset -q activity-state.json

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions