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

Удаляет gulp из сборки #369

Merged
merged 2 commits into from
Aug 8, 2024
Merged

Удаляет gulp из сборки #369

merged 2 commits into from
Aug 8, 2024

Conversation

monochromer
Copy link
Member

Fix: #368.

Заметил, что больше не используется cache busting, когда в название файла добавляется хэш типа styles.[hash].css, поэтому решил не использовать bundle plugin.

Правки:

  • Удаление gulp. Вместо него созданы custom extensions для CSS и клиентского JavaScript.
  • Копирование статических файлов также осуществляется с помощью Eleventy addPassthroughCopy. Чуть подправлен конфиг, чтобы не копировались специфичные для Eleventy файлы - *.11ty.js и *11tydata.js.
  • Вместо PostCSS используется LightningCSS. По необходимости нужно нужные фичи.
  • Вместо Rollup и Babel используется EsBuild.
  • Сборка происходит всегда, как в dev-режиме, так и в prod-режиме. Отличие только в минификации. Поэтому был удалён атрибут type="module" для подключения скриптов.

Внутри custom extensions browserslist подтягивает данные из поля browserslist файла package.json. Или можно передать явно через опцию targets каждого плагина.

Copy link
Member

@pepelsbey pepelsbey left a comment

Choose a reason for hiding this comment

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

Проверил, всё работает красиво для разработки и прода :) Спасибо!

Заметил только в JS кириллицу закодировало непривычно:

- e.innerText="Скопировать ссылку"
+ e.innerText="\u0421\u043A\u043E\u043F\u0438\u0440\u043E\u0432\u0430\u0442\u044C \u0441\u0441\u044B\u043B\u043A\u0443"

Я сейчас на всякий случай настроил сервер, чтобы он посылал всё в явном UTF-8, так что думаю безопасно будет не кодировать.

@monochromer
Copy link
Member Author

Заметил только в JS кириллицу закодировало непривычно:

надо esbuild поковырять.

@alexeyten
Copy link
Contributor

Заметил только в JS кириллицу закодировало непривычно:

надо esbuild поковырять.

https://esbuild.github.io/api/#charset

@monochromer
Copy link
Member Author

надо esbuild поковырять.

поправил

Copy link
Member

@pepelsbey pepelsbey left a comment

Choose a reason for hiding this comment

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

Огонь! Спасибо :)

@pepelsbey pepelsbey merged commit 893510a into web-standards-ru:main Aug 8, 2024
5 checks passed
pepelsbey pushed a commit that referenced this pull request Aug 8, 2024
* Удаляет gulp из сборки

* Явно задаёт кодировку `utf-8` для esbuild
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.

Избавиться от Gulp и внедрить всё в сборку Eleventy
3 participants