Skip to content
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.

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

Added the HMCTS Logging along with Lombok for the Fees Register App p… #393

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

pats-john
Copy link
Contributor

…roject.......

Before creating a pull request make sure that:

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:

JIRA link (if applicable)

Change description

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ ] No

@codecov
Copy link

codecov bot commented Jul 19, 2021

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.57%. Comparing base (d9d1218) to head (d11b0a4).
Report is 620 commits behind head on master.

Current head d11b0a4 differs from pull request most recent head 535ed16

Please upload reports for the commit 535ed16 to get more accurate results.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #393      +/-   ##
============================================
+ Coverage     72.38%   72.57%   +0.19%     
- Complexity      499      501       +2     
============================================
  Files           123      123              
  Lines          1575     1575              
  Branches        119      119              
============================================
+ Hits           1140     1143       +3     
+ Misses          362      359       -3     
  Partials         73       73              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

vianvi
vianvi previously approved these changes Jul 19, 2021
YasharRahvar
YasharRahvar previously approved these changes Jul 19, 2021
satyachundur
satyachundur previously approved these changes Jul 19, 2021
Copy link

@satyachundur satyachundur left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@hmcts-jenkins-feepay hmcts-jenkins-feepay bot temporarily deployed to preview July 19, 2021 15:57 Inactive
@hmcts-jenkins-feepay hmcts-jenkins-feepay bot temporarily deployed to preview July 21, 2021 06:12 Inactive
@hmcts-jenkins-feepay hmcts-jenkins-feepay bot temporarily deployed to preview July 22, 2021 14:54 Inactive
Copy link

github-actions bot commented May 1, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Copy link

github-actions bot commented Jul 2, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Copy link

github-actions bot commented Sep 2, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

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

Successfully merging this pull request may close these issues.

None yet

7 participants