Back to all reviewers

Parameterize build processes

mui/material-ui
Based on 2 comments
Json

When designing build scripts for multi-package repositories, use parameterization to handle special cases rather than creating duplicate scripts with minor variations. This approach maintains consistency while accommodating package-specific needs. For example, instead of hardcoding different behaviors, add flags that can modify behavior:

CI/CD Json

Reviewer Prompt

When designing build scripts for multi-package repositories, use parameterization to handle special cases rather than creating duplicate scripts with minor variations. This approach maintains consistency while accommodating package-specific needs. For example, instead of hardcoding different behaviors, add flags that can modify behavior:

"scripts": {
  "build": "standard-build-script --skipEsmPkg",
  "build:esm-pkg": "node ./scripts/create-esm-package-json.mjs"
}

Similarly, prefer built-in parameterization of tools (like pnpm’s --dry-run flag) over custom implementations for common CI/CD operations. This improves maintainability and leverages well-tested functionality.

2
Comments Analyzed
Json
Primary Language
CI/CD
Category

Source Discussions