Skip to content
This repository was archived by the owner on Dec 3, 2020. It is now read-only.

Pull request template #2

Merged
merged 1 commit into from
Oct 4, 2019
Merged

Pull request template #2

merged 1 commit into from
Oct 4, 2019

Conversation

parssinen
Copy link

  • A task has been created for the PR on the Kanban board with necessary details filled (one task / repo)
  • The commits and commit messages adhere to version control conventions
  • The code is consistent with the existing code base
  • Tests have been written for the change
  • All tests pass (unit, e2e)
  • All code has been linted and there aren't any lint errors
  • The change has been tested in the browser with Firefox 59+, Chrome 66+
  • The change has been tested with a smaller screen (tablet)
  • The change conforms to the UX specifications
  • All translations have been added (fi, sv, en)
  • The code is self-documenting or has been documented sufficiently, e.g., in the README
  • The branch has been rebased against master before the PR was created

Checklist for pull request reviewer (copy to review text box)

- [ ] A task has been created for the PR on the Kanban board with necessary details filled (one task / repo)
- [ ] The commits and commit messages adhere to [version control conventions](https://voltti.atlassian.net/wiki/spaces/NUORA/pages/32999/Versionhallintak+yt+nn+t#Versionhallintak%C3%A4yt%C3%A4nn%C3%B6t-Commit-viestienmuotoilu)
- [ ] The code is consistent with the existing code base
- [ ] All changes in all changed files have been reviewed
- [ ] Tests have been written for the change
- [ ] All tests pass (unit, e2e)
- [ ] All code has been linted and there aren't any lint errors
- [ ] The change has been tested in the browser with Firefox 59+, Chrome 66+
- [ ] The change has been tested with a smaller screen (tablet)
- [ ] The change conforms to the [UX specifications](https://voltti.atlassian.net/wiki/spaces/NUORA/pages/74809369/UX)
- [ ] All translations have been added (fi, sv, en)
- [ ] The code is self-documenting or has been documented sufficiently, e.g., in the README
- [ ] The PR branch has been rebased against master and force pushed if necessary before merging

Copy link

@kuosmark kuosmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • A task has been created for the PR on the Kanban board with necessary details filled (one task / repo)
  • The commits and commit messages adhere to version control conventions
  • The code is consistent with the existing code base
  • All changes in all changed files have been reviewed
    - [ ] Tests have been written for the change
    - [ ] All tests pass (unit, e2e)
    - [ ] All code has been linted and there aren't any lint errors
    - [ ] The change has been tested in the browser with Firefox 59+, Chrome 66+
    - [ ] The change has been tested with a smaller screen (tablet)
    - [ ] The change conforms to the UX specifications
    - [ ] All translations have been added (fi, sv, en)
  • The code is self-documenting or has been documented sufficiently, e.g., in the README
  • The PR branch has been rebased against master and force pushed if necessary before merging

@parssinen parssinen merged commit a448abd into master Oct 4, 2019
@kuosmark kuosmark deleted the pr-template branch October 7, 2019 14:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants