Make Pipeline settings generation reusable for consuming projects.

Review Request #11367 — Created Jan. 11, 2021 and submitted

Information

Djblets
release-2.x

Reviewers

We use a fairly custom Pipeline configuration for Djblets static media,
making use of wrappers for the LessCSS compiler and a standard set of
options and plugins for Babel and LessCSS. Consuming applications
(namely Review Board) use the same options, and have previously had to
duplicate the configuration. This meant that they didn't gain any
benefits from improvements made to this configuration.

This change introduces a new djblets.pipeline.settings module, which
contains a build_pipeline_settings() function that builds a standard
PIPELINE settings dictionary. It provides the standard settings we
use, but allows a degree of customization. Consumers can make use of
this directly, or modify the generated settings after the fact.

Built a Djblets package successfully.

Unit tests pass.

Made use of this in Review Board and verified the resulting static media.

Summary ID
Make Pipeline settings generation reusable for consuming projects.
We use a fairly custom Pipeline configuration for Djblets static media, making use of wrappers for the LessCSS compiler and a standard set of options and plugins for Babel and LessCSS. Consuming applications (namely Review Board) use the same options, and have previously had to duplicate the configuration. This meant that they didn't gain any benefits from improvements made to this configuration. This change introduces a new `djblets.pipeline.settings` module, which contains a `build_pipeline_settings()` function that builds a standard `PIPELINE` settings dictionary. It provides the standard settings we use, but allows a degree of customization. Consumers can make use of this directly, or modify the generated settings after the fact.
a7fae6979f08d456cd660eaac662e90a4f979440
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.x (a848a53)
Loading...