Skip to content

Add job-specific logging#1996

Merged
ArtOfCode- merged 9 commits intodevelopfrom
art/job-logs
Feb 27, 2026
Merged

Add job-specific logging#1996
ArtOfCode- merged 9 commits intodevelopfrom
art/job-logs

Conversation

@ArtOfCode-
Copy link
Member

Related to #1995.

Adds a job-specific logger tagged with the job class and randomised ID, and adds logging to our existing jobs.

@ArtOfCode- ArtOfCode- marked this pull request as ready for review February 27, 2026 14:57
@codecov
Copy link

codecov bot commented Feb 27, 2026

Codecov Report

❌ Patch coverage is 55.00000% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.96%. Comparing base (0e88121) to head (f944c17).
⚠️ Report is 10 commits behind head on develop.

Files with missing lines Patch % Lines
app/jobs/cleanup_votes_job.rb 0.00% 12 Missing ⚠️
app/jobs/clean_up_new_thread_followers_job.rb 40.00% 3 Missing ⚠️
app/jobs/clean_up_thread_followers_job.rb 40.00% 3 Missing ⚠️
Additional details and impacted files
Components Coverage Δ
controllers 75.57% <ø> (-0.04%) ⬇️
helpers 85.08% <ø> (ø)
jobs 81.18% <55.00%> (-0.30%) ⬇️
models 93.03% <ø> (ø)
tasks 61.11% <ø> (ø)

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ArtOfCode- ArtOfCode- requested a review from Oaphi February 27, 2026 14:57
Copy link
Member

@Oaphi Oaphi left a comment

Choose a reason for hiding this comment

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

LGTM - not worried about coverage here, it's mostly because we barely have any for jobs in the first place.

@ArtOfCode- ArtOfCode- merged commit 84435d4 into develop Feb 27, 2026
12 of 13 checks passed
@ArtOfCode- ArtOfCode- deleted the art/job-logs branch February 27, 2026 17:01
@ArtOfCode- ArtOfCode- added this to the v0.12.4 milestone Feb 27, 2026
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.

3 participants