Skip to content

Fix devin-review workflow TTY requirement in CI #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mnm-matin merged 3 commits into from
Feb 16, 2026
Merged

Fix devin-review workflow TTY requirement in CI #22

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
65f96bc
Initial plan
Copilot Feb 16, 2026
1b3e041
Fix devin-review workflow TTY error by emulating terminal
Copilot Feb 16, 2026
4f93ea9
Improve workflow: use env block for CI variable
Copilot Feb 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion .github/workflows/devin-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,4 +20,10 @@ jobs:
node-version: '20'

- name: Run Devin Review
run: npx devin-review ${{ github.event.pull_request.html_url }}
# Use script to emulate a TTY as devin-review requires terminal features
# The -q flag suppresses script output, -e exits with command exit code,
# and -c runs the command. /dev/null discards script's own output.
env:
CI: true # Ensures the tool runs in non-interactive CI mode
run: |
script -q -e -c "npx devin-review ${{ github.event.pull_request.html_url }}" /dev/null
Loading