-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
astro-compress middleware #131
Comments
Yeah, you can do: import compress from "astro-compress";
import { defineConfig } from "astro/config";
export default defineConfig({
integrations: [
import.meta.env.MODE === "production" ? compress() : null,
],
}); But the integration is not enabled in development it's only turned on |
Apologies you meant, |
It's useful to know that you can turn off an integration like that, but it's not specific to |
Or maybe they have an environment where |
But he writes |
Just a preference, I think, but if you put |
But even if I put |
Yes, but that is regardless of the development mode, if you put |
Oh, I see so turning it on in development would be a separate feature because in development mode it is not used anyway (right now). So in development mode, it doesn't matter what options are passed. |
Exactly! |
The current way it works is just runs after build is done over all files. |
The new way would be to attach a middleware to an internal function inside |
I see. Nice. Can you link that issue in your backlog? |
Sure. 😁 |
So can you link that issue in your backlog? |
@kireerik It's already linked, you'll get notified once it's done. 😁 The backlog is my personal task list. |
related issue:
The text was updated successfully, but these errors were encountered: