Skip to content

Conversation

@r3cha
Copy link
Contributor

@r3cha r3cha commented Mar 12, 2025

This PR as result of review and discussion here #165

@justinko
Copy link

Yup. Without this change a default rails app will break on Render.

@rafaelfranca rafaelfranca merged commit 466cd57 into rails:main Jul 10, 2025
@inkstak
Copy link

inkstak commented Aug 13, 2025

Our SASS provider pre-installs bun by default while we are using yarn in development.

For years, we commited and pushed package.json and yarn.lock and some commands depends on yarn :

  "scripts": {
    "build:css:application": "postcss ./app/assets/stylesheets/application.postcss.css -o ./app/assets/builds/application.css",
    "build:css:mailer": "postcss ./app/assets/stylesheets/mailer.postcss.css -o ./app/assets/builds/mailer.css",
    "build:css": "yarn build:css:application && yarn build:css:mailer",

Yet In production, cssbundling-rails decided to use bun over yarn to call build:css and it wasn't a problem so far.

But yesterday they upgraded bun to 1.2.20.
It includes this change : https://bun.sh/blog/bun-v1.2.20#automatic-yarn-lock-migration

Now calling bun build:css failed because it cannot found postcss command.
We worked around by overriding rake tasks from cssbundling-rails to force them to use yarn until the release of this patch.

@justinko
Copy link

I just switched to Bun ... goodbye esbuild and node.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants