What are the Slack Archives?

Itโ€™s a history of our time together in the Slack Community! Thereโ€™s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you wonโ€™t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

Hi everyone ๐Ÿ‘‹, our deployment process takes up to 400 - 500 seconds only on command 'frontend:p

U019JM2HE7N
U019JM2HE7N Posts: 61 ๐Ÿง‘๐Ÿปโ€๐Ÿš€ - Cadet

Hi everyone ๐Ÿ‘‹,
our deployment process takes up to 400 - 500 seconds only on command 'frontend:project:install-dependencies'.
It seems that we lose a lot of time for the node-gyp rebuild task there. node-gyp rebuild therefore is only necessary for โ€˜node-sassโ€™.
ย 
And 'node-sass' seems to be considered as deprecated (https://github.com/sass/node-sass/issues/2952) and can be replaced by 'dart-sass' (https://github.com/sass/dart-sass), like it is already recommended on webpacks documentation (https://webpack.js.org/loaders/sass-loader/).

While we can easily replace it on Pyz/Yves level, we want to avoid to replace it in 'spryker/oryx-for-zed' ourselves. Are there any considerations to replace โ€˜node-sassโ€™ with โ€™dart-sassโ€™ in this package? Maybe @UN6T8SEM7?

Any other recommendations to improve performance on this particular deployment step?

Comments

  • Aleksander Kovalenko
    Aleksander Kovalenko Sprykee Posts: 45 ๐Ÿง‘๐Ÿปโ€๐Ÿš€ - Cadet

    Hi Marcel! Yes we have plans to replace node-sass with dart-sass together with update to webpack 5. But I think we can even do it separately as a perfromance improvement for our build.

  • Aleksander Kovalenko
    Aleksander Kovalenko Sprykee Posts: 45 ๐Ÿง‘๐Ÿปโ€๐Ÿš€ - Cadet

    Can you please create a ticket in our support system?

  • U019JM2HE7N
    U019JM2HE7N Posts: 61 ๐Ÿง‘๐Ÿปโ€๐Ÿš€ - Cadet

    Hi Alexander! Thanks for your response. I will create a ticket ๐Ÿ‘

  • UQ958DA4U
    UQ958DA4U Posts: 232 ๐Ÿง‘๐Ÿปโ€๐Ÿš€ - Cadet
    edited January 2021

    A public announcement would be great if this is implemented.

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 โœจ - Novice
    edited January 2021

    I will add an internal note to announce it here, in the community.