Skip to content

fix: respect DAYS_BACK env var in backfill-duplicate-comments script#44393

Open
Dogecito42 wants to merge 1 commit intoanthropics:mainfrom
Dogecito42:main
Open

fix: respect DAYS_BACK env var in backfill-duplicate-comments script#44393
Dogecito42 wants to merge 1 commit intoanthropics:mainfrom
Dogecito42:main

Conversation

@Dogecito42
Copy link
Copy Markdown

The workflow passes DAYS_BACK as an environment variable but the script was ignoring it entirely, always falling back to MIN/MAX_ISSUE_NUMBER.

This fix reads DAYS_BACK to compute a cutoff date and filters issues by creation date, with early pagination termination when pages fall before the cutoff. The usage docs are also updated to document all env vars.

The workflow passes DAYS_BACK as an environment variable but the script
was ignoring it entirely, always falling back to MIN/MAX_ISSUE_NUMBER.

This fix reads DAYS_BACK to compute a cutoff date and filters issues by creation date, with early pagination termination when pages fall before the cutoff. The usage docs are also updated to document all env vars.
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