Skip to content

Follow ups from !100386

🚌 Context

In Add bulk destroy mutation for Packages (!100386 - merged), we added a GraphQL endpoint to destroy packages from the Package Registry in bulk.

During the backend maintainer review, there were a few points that were not addressed and left for a follow up:

🤔 What does this MR do and why?

  • Replace an match_array with an eq in a spec.
  • Rename Packages::Package scopes that preload associations.

The above are straightforward code cleanups and there is 0 impact on existing features. As such, a changelog was not included.

📺 Screenshots or screen recordings

See !100386 (merged)

🗄 How to set up and validate locally

See !100386 (merged)

🚓 MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by David Fernandez

Merge request reports

Loading