Skip to content

chore: Remove cyclic imports

  • Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA

Description

This removes several cyclic imports which were manually ignored by the compiler.

  • Importing only the type definition instead of requiring the module will avoid all issues that arise from cyclic dependencies during runtime.
  • JobLogRefresher has been changed to accept a JobLogCache parameter, instead of relying on the singleton.

What CHANGELOG entry will this MR create?

  • fix: Bug fix fixes - a user-facing issue in production - included in changelog
  • feature: New feature - a user-facing change which adds functionality - included in changelog
  • BREAKING CHANGE: (fix or feature that would cause existing functionality to change) - should bump major version, mentioned in the changelog
  • None - other non-user-facing changes
Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading