...
In both cases the code should be part of a repository managed using git. Typically the code should be stored in the https://bitbucket.renalregistry.nhs.uk repository. All code should be formatted using black (https://github.com/psf/black) it is recommended that this be done as a pre-save function in the preferred code editor vscode (https://code.visualstudio.com/) or a git pre-commit hook.
Development of code should be done using branches (where ever possible) and pull requests which should be reviewed by at least two developers before merging in. If an issue is identified in the code it needs fixing not ignoring and a ticket raised before merging and deploying.
Changes to code should always be supported with tests especially if they are bug fixes. One of the reasons for this is due to the history of some of the code bases it is difficult to tell if a fix has broken some other dependency so by ensuring the right tests are in place future changes hopefully will not break earlier code.
...
All code needs to be adequately tested prior to releasing to others for testing. For the UKRDC this means the developer should test the code they develop and ensure it is working as they expect then deploy to the dev server and test with some sample dataflows. Once this has been proven successful the code should be deployed to staging where a wide range of real files can be tested and wider testing done before finally releasing to live.
All code changes should be reviewed by at least two other developers before being merged in to the master branch. Typically this would be done with pull requests.
Ideally all code should be added to the CI engine Bamboo (https://www.atlassian.com/software/bamboo) and deployments set up as appropriate. Due to the way the engine can work it is important to review the logs for errors as sometimes errors do not cause a build failure. This may indicate the build process needs tweaking.
...