Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change Support app to work with dart-sass #714

Merged
merged 1 commit into from
Jan 24, 2024

Conversation

floehopper
Copy link
Contributor

@floehopper floehopper commented Jan 22, 2024

When I moved the Support app from libsass to dart-sass following these instructions, I created a new bin/dev file which runs two processes in Procfile.dev, one of which runs the rails server and another which runs dart-sass in watch mode. This updates the docker-compose.yml file accordingly.

When I moved [1] the support app from libsass to dart-sass following
these instructions [2], I created a new `bin/dev` file which runs two
processes in `Procfile.dev`, one of which runs the rails server and
another which runs `dart-sass` in watch mode. This updates the
`docker-compose.yml` file accordingly.

[1]: alphagov/support#1253
[2]: https://docs.publishing.service.gov.uk/manual/migrate-to-dart-sass-from-libsass.html
@floehopper floehopper force-pushed the change-support-app-to-work-with-dart-sass branch from 10e38f7 to a83b920 Compare January 22, 2024 16:35
Copy link
Contributor

@chrisroos chrisroos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, @floehopper 👍

@floehopper floehopper marked this pull request as ready for review January 24, 2024 12:30
@floehopper floehopper merged commit 47c5f91 into main Jan 24, 2024
2 checks passed
@floehopper floehopper deleted the change-support-app-to-work-with-dart-sass branch January 24, 2024 12:31
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.

2 participants