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

refactor: remove PR number from displayed info #75

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

Conversation

devin-ai-integration[bot]
Copy link

Remove PR number from displayed information

This PR removes the PR number from the displayed information in the browser extension UI while preserving other metadata (opened time and author).

Changes:

  • Removed PR number from the subdescription text in html-generator.js
  • Added test to verify PR number is not displayed
  • Kept all other PR information intact (opened time, author, assignee)
  • No changes to internal PR identification or storage

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

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 28 commits January 24, 2025 13:04
…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]>
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