Skip to content

push#390

Merged
Cheggin merged 1 commit into
mainfrom
reagan/update-contributing
May 4, 2026
Merged

push#390
Cheggin merged 1 commit into
mainfrom
reagan/update-contributing

Conversation

@Cheggin
Copy link
Copy Markdown
Collaborator

@Cheggin Cheggin commented May 4, 2026

Summary

Test plan

  • npm run lint passes
  • npm run typecheck passes
  • npm run test passes (unit + integration)
  • New code has tests (per D1 directive — TDD, ≥80% coverage on new src/main/src/shared modules)
  • Python changes: pytest + ruff check + mypy pass
  • Manual QA in npm run dev for UI-facing changes

Screenshots / recordings

Risk + rollback


Summary by cubic

Simplifies and modernizes CONTRIBUTING.md with a short, Task-based setup and workflow for Browser Use Desktop. Removes outdated Python/daemon content, adds clear PR/bug/feature guidance, and documents logging and build commands.

  • Refactors
    • Switches setup to task up with prerequisites: Node.js 22 and yarn.
    • Consolidates dev commands: task --list, task lint, task typecheck, cd app && yarn test, task make, and Linux builds via task linux:make:docker.
    • Adds bug report checklist plus log paths and task logs:* helpers.
    • Adds concise PR guidelines, feature request tips, and support links (Discord/Twitter).

Written for commit c9ffa33. Summary will update on new commits.

@Cheggin Cheggin requested a review from sauravpanda as a code owner May 4, 2026 08:15
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@Cheggin Cheggin merged commit a20caf5 into main May 4, 2026
8 checks passed
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