Skip to content

Conversation

@Rongbin99
Copy link
Owner

PR CHECKLIST

  • My code follows the style guidelines of this project (variable naming, commenting, copyright, etc.)
  • I have performed a self-review of my code
  • Changes are clearly highlighted and easy to understand
  • I have commented my code, particularly in hard-to-understand areas
  • I have meaningful commit messages that explain what was changed/committed
  • I have built and locally tested my changes
  • My changes generate no new errors or regressions (pending verification)
  • I have made corresponding changes to the documentation OR this is N/A
  • Documentation accurately reflects the current state of the project OR this is N/A
  • I have added tests that prove my fix is effective or that my feature works OR this is N/A
  • New and existing unit tests pass locally with my changes OR this is N/A
  • Any dependent changes have been merged and published in downstream modules OR this is N/A
  • All links are working and correct OR this is N/A
  • Spelling and grammar are correct
  • I have added the "READY FOR REVIEWS" tag when this PR is ready for reviews

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants