Skip to content

Conversation

@JackPlowman
Copy link
Contributor

@JackPlowman JackPlowman commented May 12, 2023

Task Branch Pull Request

https://nhsd-jira.digital.nhs.uk/browse/DS-780

Description of Changes

This PR updates all files in the Application folder to meet Ruff standards.

@JackPlowman JackPlowman force-pushed the task/DS-780_Application_Ruff_Implementation branch from 0335476 to c17aa8e Compare May 15, 2023 11:12
@JackPlowman JackPlowman changed the title DS-780 Apply Ruff to application folder DS-780 Apply Ruff to Application Folder May 15, 2023
@JackPlowman JackPlowman marked this pull request as ready for review May 15, 2023 13:07
@JackPlowman JackPlowman requested a review from a team as a code owner May 15, 2023 13:07
@JackPlowman JackPlowman enabled auto-merge (squash) May 15, 2023 13:24
@JackPlowman JackPlowman force-pushed the task/DS-780_Application_Ruff_Implementation branch from 7c6754d to 384a531 Compare May 16, 2023 07:00
@JackPlowman JackPlowman disabled auto-merge May 16, 2023 12:41
Copy link
Collaborator

@mabe13 mabe13 left a comment

Choose a reason for hiding this comment

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

Added two comments about checking a commented function and it's test are meant to be deleted. Another than that happy for this one to be merged

@sonarqubecloud
Copy link

sonarqubecloud bot commented May 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

96.3% 96.3% Coverage
0.0% 0.0% Duplication

@JackPlowman JackPlowman enabled auto-merge (squash) May 17, 2023 08:40
@JackPlowman JackPlowman merged commit 88fa4de into develop May 17, 2023
@JackPlowman JackPlowman deleted the task/DS-780_Application_Ruff_Implementation branch May 17, 2023 09:06
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.

2 participants