fix(bundle): remove rebuild because it causes slow performance on big…

[BorjaMacedo]

When the project is large, rebuild causes memory problems and prevents deployment in github actions and other environments.

[jamesmhaley]

+1 This is also blocking our pipeline. Would be amazing to get this into this amazing project!!

[floydspace]

Hi @BorjaMacedo thanks for the PR...
but we need to be careful here, I see in the changes history we first introduced the rebuild then removed it, then again added, and now removing again:

https://github.com/floydspace/serverless-esbuild/pull/487/files
https://github.com/floydspace/serverless-esbuild/pull/491/files

I feel like better we need to introduce a new config option skipRebuild (or disableIncremental like we had it before v1.48.0), so everyone is happy.

Is it what you can implement in this PR?

[BorjaMacedo]

Hi @BorjaMacedo thanks for the PR... but we need to be careful here, I see in the changes history we first introduced the rebuild then removed it, then again added, and now removing again:

https://github.com/floydspace/serverless-esbuild/pull/487/files https://github.com/floydspace/serverless-esbuild/pull/491/files

I feel like better we need to introduce a new config option skipRebuild (or disableIncremental like we had it before v1.48.0), so everyone is happy.

Is it what you can implement in this PR?

I have implemented it and it seems to work perfectly. If you need me to do something else I can do it.

[floydspace]

hey @BorjaMacedo , looks good, could you please also add the new property in readme options table with a short description what it does, and I will merge it.

[BorjaMacedo]

@floydspace Done my friend! ;)

[floydspace]

@BorjaMacedo awesome thank you, merging it

[github-actions]

🎉 This PR is included in version 1.52.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀