Skip to content

Auto Backfilling#3

Open
esphas wants to merge 9 commits into
nornagon:actionfrom
esphas:backfilling
Open

Auto Backfilling#3
esphas wants to merge 9 commits into
nornagon:actionfrom
esphas:backfilling

Conversation

@esphas

@esphas esphas commented Aug 27, 2025

Copy link
Copy Markdown
Contributor

this PR addresses the backfilling issue mentioned in #2 by implementing the following changes:

  • adds a version field to builds in builds.json, which will be recorded starting with this change (const buildVersion = 1;)
  • on each run, the action will now identify previous builds with a lower version than the current one and perform backfilling (up to the BACKFILL_LIMIT)

@esphas esphas marked this pull request as draft August 27, 2025 02:36
@esphas esphas marked this pull request as ready for review August 27, 2025 06:16

@nornagon nornagon left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

like the approach! couple comments

Comment thread pull-data.mjs
Comment thread pull-data.mjs
@esphas

esphas commented Mar 23, 2026

Copy link
Copy Markdown
Contributor Author

@nornagon hi! just wanted to follow up on this PR. I noticed there have been some upstream updates that caused conflicts, so I merged the latest changes into my branch and resolved them.
just wanted to check if you're still planning to merge this, or if there's anything else you'd like me to adjust.

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.

2 participants