Skip to content

Allow configurable generated-files builder timeout - #1313

Merged
yakew7 merged 2 commits into
yakew7:mainfrom
renuvenugopal178:renuvenugopal178-patch-1
Nov 9, 2026
Merged

yakew7 merged 2 commits into
yakew7:mainfrom
renuvenugopal178:renuvenugopal178-patch-1

Conversation

@renuvenugopal178

Copy link
Copy Markdown
Contributor

Summary

  • Allow the generated-files builder timeout to be configured using the FAIRCODE_BUILD_TIMEOUT environment variable.
  • Default to 600 seconds when the value is missing, invalid, or non-positive.
  • Add tests covering valid, missing, and invalid timeout values.

Testing

  • Added automated tests for timeout configuration behavior.

Added functionality to configure regeneration timeout via FAIRCODE_BUILD_TIMEOUT environment variable.
@vercel

vercel Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

@renuvenugopal178 is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

Hey @renuvenugopal178, your first PR on Fair Code, that's awesome, thank you.

This project is about making AI more accountable, and contributions like yours are what keep that work going. We'll review your changes shortly.

If you haven't already, give the contributing guide a quick read: it covers how audits are structured and what we look for in a review.

And if you're finding Fair Code useful or interesting, a star on the repo genuinely helps more people find it - no pressure, just appreciated.

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

@yakew7 @ahmdkaml @propcgamer20-png - new PR touching a path you own, please review.

@yakew7
yakew7 merged commit a2d06ca into yakew7:main Nov 9, 2026
19 of 20 checks passed
yakew7 added a commit that referenced this pull request Nov 11, 2026
Started in one run with check_contributors.py --stub ... --commits (snapshot
moved to PR #1313, the pull request summaries filled in the stubs) and the
TODO sentences replaced: the pending-closes draft note (#1310), empty groups
in significance (#1311) and the configurable builder timeout (#1313). PR
#1311's commit is authored by PRIYANKA2272004, who has no pull request of
her own, so ranjanasri6's commit count and PRIYANKA2272004 are documented
exceptions, and an identity map row is added for renuvenugopal178.
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