Skip to content

Conversation

@martinboulais
Copy link
Contributor

I have a JIRA ticket

  • branch and/or PR name(s) include(s) JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected

Notable changes for users:

  • N/A

Notable changes for developers:

  • Added more logs to external service synchronization (CCDB, MonALISA)

Changes made to the database:

  • N/A

@codecov
Copy link

codecov bot commented Dec 13, 2024

Codecov Report

Attention: Patch coverage is 87.50000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 44.08%. Comparing base (5dafb0e) to head (34057c3).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
lib/application.js 0.00% 1 Missing ⚠️
...alServicesSynchronization/ccdb/CcdbSynchronizer.js 91.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1818      +/-   ##
==========================================
+ Coverage   44.04%   44.08%   +0.03%     
==========================================
  Files         889      889              
  Lines       15856    15862       +6     
  Branches     2990     2990              
==========================================
+ Hits         6984     6992       +8     
+ Misses       8872     8870       -2     

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

@martinboulais martinboulais force-pushed the mboulais/O2B-1417/add-more-log-to-services-sync branch from c21faf9 to 6eaf6bf Compare December 13, 2024 11:27
@martinboulais martinboulais force-pushed the mboulais/O2B-1417/add-more-log-to-services-sync branch from 6eaf6bf to 34057c3 Compare December 16, 2024 07:46
@martinboulais martinboulais merged commit 83ddb13 into main Dec 17, 2024
23 checks passed
@martinboulais martinboulais deleted the mboulais/O2B-1417/add-more-log-to-services-sync branch December 17, 2024 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants