JeremyYao commented on code in PR #1575: URL: https://github.com/apache/daffodil-vscode/pull/1575#discussion_r2686735598
########## DEVELOPMENT.md: ########## @@ -210,6 +217,55 @@ Once changes are pushed, you can make pull requests with completed changes back  +### Pull Request Approval and Merge Process + +Once you've created a pull request to the main daffodil-vscode branch, it must go through the Apache Daffodil project's approval process before it can be merged. + +#### Approval Requirements + +**All pull requests require at least two +1 votes (green approvals) from PMC members or committers before they can be merged.** This ensures that changes are properly reviewed and meet the project's quality standards. The approvals must come from committers with write access to the repository. + +#### Review Cycle + +The review process typically follows this workflow: + +1. **Submit Pull Request**: Create your PR with your proposed changes to the main branch +2. **Reviewer Feedback**: PMC members, committers, and other contributors will review your code and provide feedback +3. **Address Feedback**: Add new commits to address review comments + - **Important**: Do not squash previously reviewed commits during the review cycle + - Each review iteration should involve a separate, distinct commit to preserve the review commentary + - The pull request will automatically update with new commits pushed to your branch +4. **Approval**: Once reviewers are satisfied, they will provide +1 votes (approvals) +5. **Merge**: After receiving at least two +1 votes from PMC members or committers, your PR is ready to be merged + +#### Merging Process + +Only committers with merged GitHub and ASF (Apache Software Foundation) accounts have the authority to merge pull requests. + +**Merge Method**: When merging, committers must: +- Click the "Merge pull request" dropdown +- Select **"Rebase and merge"** +- Do **not** use "Squash and merge" or "Create a merge commit" + +This maintains a clean, linear commit history while preserving individual commits from the pull request. + +#### Post-Merge Actions + +After a pull request is merged, the committer who performed the merge should: +1. Mark the associated GitHub issue as "Closed" or move it to the appropriate project board status +2. Add a comment with the git commit hash from the merge for traceability + +#### Before Starting Work + +To signal to the community that you are actively working on an issue and avoid duplicate efforts: +- **Assign the GitHub issue to yourself** before beginning development +- Add a comment on the issue to let others know you're working on it +- If the issue is part of a milestone project board, move it to "In Progress" + +#### Additional Resources + +For more detailed information about the Apache Daffodil contribution workflow, including JIRA integration and other Apache-specific processes, please refer to the [Apache Daffodil Code Contributor Workflow](https://cwiki.apache.org/confluence/display/DAFFODIL/Code+Contributor+Workflow) documentation. Review Comment: Looks good. Will mark this as resolved. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
