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

Added Pull Request Template #1812

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Rishi-source
Copy link

@Rishi-source Rishi-source commented Mar 11, 2025

This PR creates a pull request template to standardize contributions and make the review process more efficient. This template provides a clear structure for contributors to describe their changes, specify the type of change, verify they've met project requirements, and disclose any AI usage.

Signed-off-by: Rishi Garg rishigarg2503@gmail.com

@Rishi-source
Copy link
Author

Hi @pombredanne @AyanSinhaMahapatra, I have added the Pull Request Template as discussed in yesterday’s meeting. Please have a look and let me know if any changes are needed.

@Rishi-source Rishi-source force-pushed the pull_request_template branch 2 times, most recently from 1600bfd to ee5d494 Compare March 17, 2025 18:14
@TG1999
Copy link
Contributor

TG1999 commented Mar 20, 2025

@Rishi-source please use assertive statements while writing commits : ) https://www.freecodecamp.org/news/how-to-write-better-git-commit-messages/

@Rishi-source
Copy link
Author

sure @TG1999 I'll rebase all the commit into one commit according to the doc.

@Rishi-source Rishi-source force-pushed the pull_request_template branch 2 times, most recently from 82906ab to 2272157 Compare March 23, 2025 11:31
Signed-off-by: RISHI GARG <134256793+Rishi-source@users.noreply.github.com>
@Rishi-source Rishi-source force-pushed the pull_request_template branch from 65faff2 to 7c75047 Compare March 23, 2025 11:35
@Rishi-source
Copy link
Author

@TG1999 I have squashed all the commits into one and also used assertive statement in the commit message.

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