Skip to content

Remove mock dependency#16

Draft
Tom-Willemsen wants to merge 4 commits intomainfrom
remove_mock_dependency
Draft

Remove mock dependency#16
Tom-Willemsen wants to merge 4 commits intomainfrom
remove_mock_dependency

Conversation

@Tom-Willemsen
Copy link
Member

Description of work

Add your own description here

To test

ISISComputingGroup/uktena#3

Acceptance criteria

List the acceptance criteria for the PR


Code Review

  • Is the code of an acceptable quality?
  • Are there unit tests in place? Are the unit tests small and test the a class in isolation?
  • Have the changes been documented in the release notes. If so, do they describe the changes appropriately?

Functional Tests

  • Do changes function as described? Add comments below that describe the tests performed.
  • How do the changes handle unexpected situations, e.g. bad input?
  • Has developer documentation been updated if required?

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