Skip to content
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.

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

feat: add PR check status indicators #74

Open
wants to merge 92 commits into
base: feature/v2
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link

Add PR Check Status Indicators

This PR adds check status indicators (✅/❌/⏳) to pull request titles in the extension popup, similar to how GitHub displays them in PR lists.

Changes

  • Add checkConclusion to PullRequest type to track check run status
  • Fetch check runs status from GitHub API for each PR
  • Display check status emoji next to PR titles
  • Add CSS styling for check status display

Testing

  • Changes will be validated by CI
  • Manual testing can be done by checking PRs with various check statuses

Link to Devin run: https://app.devin.ai/sessions/2a30752443f2490fb55709033c6b794c

Alessandro Rodi and others added 30 commits January 21, 2025 07:49
- Convert SCSS files to CSS
- Remove SASS dependencies and configuration
- Add stylelint for CSS linting
- Update webpack config to handle CSS only
- Update imports to use .css extension

Co-Authored-By: [email protected] <[email protected]>
- Rename constants.ts to constants.js
- Remove TypeScript type annotations
- Add type declarations file (constants.d.ts)
- Update tsconfig.json to support JavaScript files

Co-Authored-By: [email protected] <[email protected]>
- Add babel-jest transform for .js files
- Configure @babel/preset-env for current Node environment
- Maintain existing ts-jest configuration for .ts files

Co-Authored-By: [email protected] <[email protected]>
…to-js

Migrate constants.ts to JavaScript
- Make Puppeteer headless mode configurable via PUPPETEER_HEADLESS env var
- Add GitHub Actions workflow with Chrome dependencies and Xvfb setup
- Configure CI environment for proper browser testing

Co-Authored-By: [email protected] <[email protected]>
- Add --no-sandbox and other required flags for CI
- Conditionally apply flags based on CI environment
- Keep default configuration for local development

Co-Authored-By: [email protected] <[email protected]>
- Set CI=true to enable CI-specific Chrome flags
- Set PUPPETEER_HEADLESS=true explicitly for CI environment
- Ensure consistent headless behavior in CI

Co-Authored-By: [email protected] <[email protected]>
- Add missing trailing commas in args array
- Fix TSLint trailing-comma errors

Co-Authored-By: [email protected] <[email protected]>
devin-ai-integration bot and others added 26 commits January 24, 2025 13:07
…d-webpack

Remove TypeScript and webpack dependencies
- Add ignored-group section with visual separation
- Filter and group ignored PRs separately from active PRs
- Style ignored section with reduced opacity and border

Co-Authored-By: [email protected] <[email protected]>
- Store category information when collecting ignored PRs
- Group ignored PRs by their original category
- Add category titles within ignored section
- Style category titles with consistent theme colors

Co-Authored-By: [email protected] <[email protected]>
- Add checkConclusion to PullRequest type
- Fetch check runs status from GitHub API
- Display check status emoji (✅/❌/⏳) next to PR titles
- Add CSS styling for check status display

Co-Authored-By: [email protected] <[email protected]>
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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