Skip to content

Minor artifacts code cleanup

Miranda Fluharty requested to merge mfluharty-prune-artifacts-page into master

What does this MR do and why?

We refactored the job status in the artifacts table to use only the <ci-icon> component, so we can remove a few things that are no longer used:

  • The succeeded line from mapBooleansToJobNodes and the JOB_STATUS_GROUP_SUCCESS constant that it depended on
    • This was used here to determine whether to show the icon or the badge
  • Remove STATUS_BADGE_VARIANTS
    • This was used here to map job status groups to their badge variants

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

No visual changes expected

How to set up and validate locally

  1. navigate to a project with a pipeline that generates artifacts
  2. in the sidebar, choose Build => Artifacts
  3. verify that the page looks and works the same as before
Edited by Miranda Fluharty

Merge request reports