- relevant information updated in
README.MD - swagger doc created/updated so that other developers can read the related documentation.
- logging included/updated so that we can debug errors when the app is live in the UAT environment.
- unit tests created/updated so that we can integrate this new release and sleep without worries at night/day.
- No merge conflict with
developbranch. - Check PR title is correct, (i.e
[AC-xx: Feature] Create pull request template). - Check your app will run without crashes.
- Check if there's hard coded values.
- No
//eslint-disable-next-line no-unused-vars, unless otherwise specified. - No unnecessary spaces between lines of code.
- Check you are using
camelCase && PascalCasecorrectly. - Check you named your code properly.
- Remove unused comments.