Skip to content

Release changing npm package upload from sync to async

Moaz Khalifa requested to merge 433009-cleanup-upload-npm-packages-async-FF into master

What does this MR do and why?

In Add worker to upload npm packages async (!146493 - merged), we changed how we upload packages to the npm Registry from sync (during the request) to async (in a background sidekiq job).

The feature was gated behind a feature flag, and it was globally enabled for about two days now without issues.

So this MR is to clean up the feature flag and release the feature in milestone 17.0 (It's a breaking change and should be release in it).

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

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Related to #433009 (closed)

Edited by Moaz Khalifa

Merge request reports