Open vsvamsi1 opened 1 day ago
[!IMPORTANT]
Review skipped
Draft detected.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The pull request introduces changes to the craco.common.config.js
file, specifically in the webpack.configure
section. A new property devtool
is added to enable source map generation, and the optimization
section is modified to include a minimize
property set to false
. The configuration for icon cache groups remains unchanged, while comments and the ignoreWarnings
array are preserved to maintain clarity regarding source maps and specific module warnings.
File | Change Summary |
---|---|
app/client/craco.common.config.js | - Added devtool: "source-map" under webpack.configure .- Updated optimization to include minimize: false .- Preserved comments in optimization.splitChunks.cacheGroups.icons and retained ignoreWarnings array. |
client-build.yml
file regarding the build process may relate to the modifications in the craco.common.config.js
file.craco.common.config.js
.is-pg-build
parameter in the workflow configuration may connect to overall build process adjustments reflected in the craco.common.config.js
.RequestTemplate
component and its testing may relate to the overall configuration and build process.skip-changelog
, ok-to-test
, Task
🎉 In the config where the webpack plays,
A newdevtool
lights up the ways.
With minimization now set to false,
Source maps guide us, like a friendly pulse.
Comments preserved, warnings ignored,
A clearer path for builds, we’ve scored! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
/build-deploy-preview skip-tests=true
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11930973095.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: true
.
env: ``.
PR: 37593.
recreate: .
/build-deploy-preview skip-tests=true
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11931296678.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: true
.
env: ``.
PR: 37593.
recreate: .
/build-deploy-preview skip-tests=true
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11931386061.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: true
.
env: ``.
PR: 37593.
recreate: .
/build-deploy-preview
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11931727846.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: . env:
.
PR: 37593.
recreate: .
/build-deploy-preview skip-tests=true
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11931766667.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: true
.
env: ``.
PR: 37593.
recreate: .
/build-deploy-preview skip-tests=true
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11931982584.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: true
.
env: ``.
PR: 37593.
recreate: .
Deploy-Preview-URL: https://ce-37593.dp.appsmith.com
/build-deploy-preview skip-tests=true
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11932919735.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: true
.
env: ``.
PR: 37593.
recreate: .
/build-deploy-preview skip-tests=true
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/11932925105.
Workflow: On demand build Docker image and deploy preview
.
skip-tests: true
.
env: ``.
PR: 37593.
recreate: .
Deploy-Preview-URL: https://ce-37593.dp.appsmith.com
Deploy-Preview-URL: https://ce-37593.dp.appsmith.com
Description
Fixes #
Issue Number
or
Fixes
Issue URL
Automation
/ok-to-test tags=""
:mag: Cypress test results
Communication
Should the DevRel and Marketing teams inform users about this change?
Summary by CodeRabbit