Skip to content

Conversation

@Chesars
Copy link

@Chesars Chesars commented Aug 18, 2025

Hi! This adds a contributing guide to help new developers to contribute to Open SWE.

- Add structured contribution guidelines following OSS best practices
- Include development setup with environment configuration
- Document code quality requirements (linting, formatting, testing)
- Add PR guidelines and community standards
- Reference official documentation for detailed setup
@vercel
Copy link

vercel bot commented Aug 18, 2025

@Chesars is attempting to deploy a commit to the LangChain Team on Vercel.

A member of the Team first needs to authorize it.

@Chesars Chesars changed the title docs: Add comprehensive CONTRIBUTING.md guide docs: Add CONTRIBUTING.md guide Aug 18, 2025
- Removed unverified commands (yarn test:int, yarn eval:single)
- Simplified structure and removed speculative policies
- Added note about format:check issues with /dist files
- Focused on essential setup steps and verified commands only
- Reduced from 175 to 81 lines while keeping all necessary information
@Chesars Chesars force-pushed the add-contributing-guide branch from c641f43 to 6bd1175 Compare August 20, 2025 21:16
Chesars and others added 5 commits August 20, 2025 19:11
- Added detailed step-by-step GitHub App creation instructions
- Included all required permissions, webhooks, and OAuth settings
- Added credential collection steps with specific locations
- Clarified GitHub App vs OAuth App differences
- Added ngrok setup for webhook URL
- Included installation requirements after app creation
- Changed Node.js requirement to "18 or higher" for future compatibility
- Added specific Yarn version requirement (3.5.1 or higher)
- Added Git as an explicit requirement
- Made requirements section more standard and clear
- Removed unnecessary note about format:check and /dist files
- Simplified lint command description
- Focused on essential information only
- Made guide more minimalist and actionable
- Add note explaining yarn build is not needed for development
- Clarify ngrok port 2024 is for agent service
- Add verification section with localhost URLs
- Mark production build as optional in workflow
Copy link
Contributor

@Palashio Palashio left a comment

Choose a reason for hiding this comment

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

thanks for this -- just one small nit

@Chesars Chesars marked this pull request as draft September 1, 2025 13:14
@Chesars Chesars marked this pull request as ready for review September 6, 2025 00:06
@Chesars Chesars marked this pull request as draft September 19, 2025 18:25
Removed detailed development setup instructions and requirements for Node.js and Yarn from the CONTRIBUTING.md file.
@Chesars Chesars changed the title docs: Add CONTRIBUTING.md guide docs: adds a guide for contributors Sep 20, 2025
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.

2 participants