Skip to content

feat: implement issue #294 — fix(dev-lead): expose ACTION_TIMEOUT_SEC repo var in workflow env - #295

Merged
don-petry merged 2 commits into
mainfrom
dev-lead/issue-294-20260520-0050
May 20, 2026
Merged

don-petry merged 2 commits into
mainfrom
dev-lead/issue-294-20260520-0050

Conversation

@don-petry

Copy link
Copy Markdown
Collaborator

Closes #294

Implemented by dev-lead agent. Please review.

Copilot AI review requested due to automatic review settings May 20, 2026 00:51
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented May 20, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@don-petry has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 55 minutes and 2 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 283db695-8cf2-42d0-b097-9cf71bf71b4d

📥 Commits

Reviewing files that changed from the base of the PR and between b43c2e2 and 47e9922.

📒 Files selected for processing (1)
  • .github/workflows/dev-lead.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev-lead/issue-294-20260520-0050

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — human-pr (no-changes)

No changes were needed for this PR.

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — fix-bot-comment (no-changes)

Engine ran but made no changes.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Exposes ACTION_TIMEOUT_SEC as a top-level workflow environment variable in the Dev-Lead workflow so engine writer invocations (used by the issue implementation path) can use the repo-configured timeout instead of the default 10 minutes.

Changes:

  • Add ACTION_TIMEOUT_SEC: ${{ vars.ACTION_TIMEOUT_SEC || '600' }} to the top-level env: block in .github/workflows/dev-lead.yml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@don-petry
don-petry enabled auto-merge (squash) May 20, 2026 00:52
@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — fix-bot-comment (no-changes)

Engine ran but made no changes.

@don-petry

Copy link
Copy Markdown
Collaborator Author

@dev-lead - please fix this PR

@sonarqubecloud

Copy link
Copy Markdown

@don-petry
don-petry merged commit a8d5c9f into main May 20, 2026
16 checks passed
@don-petry
don-petry deleted the dev-lead/issue-294-20260520-0050 branch May 20, 2026 01:48
don-petry added a commit that referenced this pull request Jun 7, 2026
… repo var in workflow env (#295)

Co-authored-by: donpetry-bot <donpetry-bot@users.noreply.github.com>
don-petry added a commit that referenced this pull request Jun 7, 2026
… repo var in workflow env (#295)

Co-authored-by: donpetry-bot <donpetry-bot@users.noreply.github.com>
don-petry added a commit that referenced this pull request Jun 8, 2026
… repo var in workflow env (#295)

Co-authored-by: donpetry-bot <donpetry-bot@users.noreply.github.com>
don-petry added a commit that referenced this pull request Jun 12, 2026
… repo var in workflow env (#295)

Co-authored-by: donpetry-bot <donpetry-bot@users.noreply.github.com>
don-petry added a commit that referenced this pull request Jun 14, 2026
… repo var in workflow env (#295)

Co-authored-by: donpetry-bot <donpetry-bot@users.noreply.github.com>
don-petry added a commit that referenced this pull request Jun 18, 2026
… repo var in workflow env (#295)

Co-authored-by: donpetry-bot <donpetry-bot@users.noreply.github.com>
don-petry added a commit that referenced this pull request Jun 25, 2026
… repo var in workflow env (#295)

Co-authored-by: donpetry-bot <donpetry-bot@users.noreply.github.com>
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.

fix(dev-lead): expose ACTION_TIMEOUT_SEC repo var in workflow env

3 participants