New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tidy features report #1380

Draft

DavidBiddle wants to merge 4 commits into main from tidy-features-report

Contributor

DavidBiddle commented Aug 13, 2024

What problem does this pull request solve?

Trello card:

Things to consider when reviewing

Ensure that you consider the wider context.
Does it work when run on your machine?
Is it clear what the code is doing?
Do the commit messages explain why the changes were made?
Are there all the unit tests needed?
Has all relevant documentation been updated?

DavidBiddle added 4 commits

August 13, 2024 09:06


          Remove redundant template argument

87e9aab


          Move fetch logic into new service

1af0841


          Move feature row formatting logic into service

0625b96


          Move table generation logic into service

6799f2a

sonarqubecloud bot commented Aug 13, 2024

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
51.0% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet