Skip to content

Conversation

bkochauri-memphis
Copy link
Collaborator

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

  • Test deploy

Checklist:

  • I have performed a self-review of my code
  • I have made corresponding changes to the knowledge base (if needed)
  • My changes generate no new warnings
  • I have verified that the specification is met and all functionalities are working as expected

Reviewer Score - 0-100%

  • Meeting Task Specifications (40%)

    • This includes security, observability, and cost-efficiency aspects.
    • Ensure that the task requirements are fully met and that the implementation aligns with the specifications provided.
  • Writing Readable Code (30%)

    • Write code that is easy to read and understand.
    • Use clear and meaningful variable names, and include comments where necessary.
  • Attention to the folder structures in automation tools and repositories (20%)

    • Make sure the folder hierarchy and the names are clear and meaningful.
  • Attention to Edge Cases (10%)

    • Identify and handle edge cases that may not be immediately obvious.
    • Demonstrate thorough testing and consideration of potential issues.

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