Routines for development
Describes routines for developing frontend code for resources in the DevOps team.
When selecting an issue from the backlog, assign yourself to the issue and set it to the Developing pipeline in the Altinn Tjenester 3.0 - DevOps Team board
App frontend
- Create a new branch from
main
.- Naming convention for branches is
bug/<issue number>-<short description>
orfeature/<issue number>-<short description>
, depending on the issue type.
- Naming convention for branches is
- Code your changes and test them by running an app locally.
- Run unit tests
- Update any failing tests
- Add new tests when needed
- Make sure you update the
version
inaltinn-app-frontend/package.json
, following semantic versioning. - Commit/push your changes and create a pull request.
- Tag the issue in the description of the PR. Do not use “fix”/“fixes” issue, as this will close the issue when PR is merged.
- Move the issue to the Review/QA pipeline in board and tag/notify any reviewers.
- Fix any review comments and/or issues that come up from running the automated pipelines.
- All pipelines should run green.
- Test coverage should be >65%.
- No new code smells should be introduced.
- Once approval of code is in and all pipelines run green, move the issue to Testing pipeline in board and unassign yourself.
- Fix any bugs that are uncovered by manual testing.
- Once testing is completed, tester will move issue to Done pipeline in board. When this is done, code can be merged into
main
.- Make sure the
version
inaltinn-app-frontend/package.json
is still updated to a new version (new code might have been merged in the mean time).
- Make sure the
- Update any relevant documentation. F.ex.
- Check if there are any tasks in the issue that are incomplete. If all tasks are completed, the issue can be closed.
Studio frontend
- Create a new branch from
main
. Naming convention for branches isbug/<issue number>-<short description>
orfeature/<issue number>-<short description>
, depending on the issue type. - Code your changes and test them by running Designer locally.
- Run unit tests
- Update any failing tests
- Add new tests when needed
- Commit/push your changes and create a pull request.
- Tag the issue in the description of the PR. Do not use “fix”/“fixes” issue, as this will close the issue when PR is merged.
- Move the issue to the Review/QA pipeline in board and tag/notify any reviewers.
- Fix any review comments and/or issues that come up from running the automated pipelines.
- All pipelines should run green.
- Test coverage should be >85%.
- No new code smells should be introduced.
- Once approval of code is in and all pipelines run green, merge the code to
main
, move the issue to Testing pipeline in board and unassign yourself. - Fix any bugs that are uncovered by manual testing.
- Update any relevant documentation. F.ex.
- Once testing is complete, check if there are any tasks in the issue that are incomplete. If all tasks are completed, the issue can be closed.