build(deps-dev): remove package rollup-plugin-license (#1796)

The `rollup-plugin-license` has been using too many deprecated dependencies, so it is necessary to remove it.

As an alternative, this changes uses Rollup `output.banner` to insert copyright information. Since `terser` runs after `output`, it is not possible to insert the Front Matter defining permlink for `sw.js` through the same way (Jekyll Front Matter is YAML rather than JS, which would cause errors with terser).

Therefore, _Jekyll Collection_ is now used to add permlink to `sw.js`, with the collection named `app`, and the directory placed in `_app`. This directory is not tracked by git, but it will be included when building the gem.
This commit is contained in:
Cotes Chung 2024-06-05 23:51:10 +08:00 committed by GitHub
parent 250880c088
commit 7ca9c59784
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
10 changed files with 35 additions and 58 deletions

1
.gitignore vendored
View file

@ -7,6 +7,7 @@ Gemfile.lock
.jekyll-cache .jekyll-cache
.jekyll-metadata .jekyll-metadata
_site _site
_app
# RubyGems # RubyGems
*.gem *.gem

View file

@ -168,6 +168,9 @@ collections:
tabs: tabs:
output: true output: true
sort_by: order sort_by: order
app:
output: true
permalink: /:name
defaults: defaults:
- scope: - scope:
@ -190,10 +193,6 @@ defaults:
values: values:
layout: page layout: page
permalink: /:title/ permalink: /:title/
- scope:
path: assets/js/dist
values:
swcache: true
sass: sass:
style: compressed style: compressed

View file

@ -6,8 +6,6 @@
<!-- layout specified --> <!-- layout specified -->
{% assign js_dist = '/assets/js/dist/' %}
{% if page.layout == 'post' or page.layout == 'page' or page.layout == 'home' %} {% if page.layout == 'post' or page.layout == 'page' or page.layout == 'home' %}
{% assign urls = urls | append: ',' | append: site.data.origin[type]['lazy-polyfill'].js %} {% assign urls = urls | append: ',' | append: site.data.origin[type]['lazy-polyfill'].js %}
@ -62,7 +60,8 @@
{% assign js = 'commons' %} {% assign js = 'commons' %}
{% endcase %} {% endcase %}
{% capture script %}{{ js_dist }}{{ js }}.min.js{% endcapture %} {% capture script %}/assets/js/dist/{{ js }}.min.js{% endcapture %}
<script src="{{ script | relative_url }}"></script> <script src="{{ script | relative_url }}"></script>
{% if page.math %} {% if page.math %}
@ -93,7 +92,7 @@
{% if jekyll.environment == 'production' %} {% if jekyll.environment == 'production' %}
<!-- PWA --> <!-- PWA -->
{% if site.pwa.enabled %} {% if site.pwa.enabled %}
<script defer src="{{ 'app.min.js' | prepend: js_dist | relative_url }}"></script> <script defer src="{{ 'app.min.js' | relative_url }}"></script>
{% endif %} {% endif %}
<!-- Web Analytics --> <!-- Web Analytics -->

View file

@ -1 +0,0 @@
Chirpy v<%= pkg.version %> | © 2019 <%= pkg.author %> | <%= pkg.license %> Licensed | <%= pkg.homepage %>

View file

@ -1,3 +0,0 @@
---
permalink: /:basename
---

View file

@ -11,7 +11,7 @@ Gem::Specification.new do |spec|
spec.license = "MIT" spec.license = "MIT"
spec.files = `git ls-files -z`.split("\x0").select { |f| spec.files = `git ls-files -z`.split("\x0").select { |f|
f.match(%r!^((_(includes|layouts|sass|(data\/(locales|origin)))|assets)\/|README|LICENSE)!i) f.match(%r!^((_(includes|layouts|sass|app|(data\/(locales|origin)))|assets)\/|README|LICENSE)!i)
} }
spec.metadata = { spec.metadata = {

View file

@ -8,6 +8,7 @@
}, },
"author": "Cotes Chung", "author": "Cotes Chung",
"license": "MIT", "license": "MIT",
"since": 2019,
"bugs": { "bugs": {
"url": "https://github.com/cotes2020/jekyll-theme-chirpy/issues" "url": "https://github.com/cotes2020/jekyll-theme-chirpy/issues"
}, },
@ -43,7 +44,6 @@
"husky": "^9.0.11", "husky": "^9.0.11",
"purgecss": "^6.0.0", "purgecss": "^6.0.0",
"rollup": "^4.18.0", "rollup": "^4.18.0",
"rollup-plugin-license": "^3.4.0",
"semantic-release": "^24.0.0", "semantic-release": "^24.0.0",
"stylelint": "^16.6.1", "stylelint": "^16.6.1",
"stylelint-config-standard-scss": "^13.1.0" "stylelint-config-standard-scss": "^13.1.0"

View file

@ -1,38 +1,43 @@
import babel from '@rollup/plugin-babel'; import babel from '@rollup/plugin-babel';
import terser from '@rollup/plugin-terser'; import terser from '@rollup/plugin-terser';
import license from 'rollup-plugin-license';
import { nodeResolve } from '@rollup/plugin-node-resolve'; import { nodeResolve } from '@rollup/plugin-node-resolve';
import fs from 'fs';
import path from 'path';
import yaml from '@rollup/plugin-yaml'; import yaml from '@rollup/plugin-yaml';
import fs from 'fs';
import pkg from './package.json';
const SRC_DEFAULT = '_javascript'; const SRC_DEFAULT = '_javascript';
const DIST_DEFAULT = 'assets/js/dist'; const DIST_DEFAULT = 'assets/js/dist';
const SRC_PWA = `${SRC_DEFAULT}/pwa`; const SRC_PWA = `${SRC_DEFAULT}/pwa`;
const DIST_PWA = '_app';
const banner = `/*!
* ${pkg.name} v${pkg.version} | © ${pkg.since} ${pkg.author} | ${pkg.license} Licensed | ${pkg.homepage}
*/`;
const isProd = process.env.BUILD === 'production'; const isProd = process.env.BUILD === 'production';
if (fs.existsSync(DIST_DEFAULT)) { function cleanup(...directories) {
fs.rm(DIST_DEFAULT, { recursive: true, force: true }, (err) => { for (const dir of directories) {
if (err) { fs.rm(dir, { recursive: true, force: true }, (err) => {
throw err; if (err) {
} console.error(`Failed to remove directory ${dir}: ${err}`);
}); }
});
}
} }
function build(filename, opts = {}) { function build(filename, opts = {}) {
const src = opts.src || SRC_DEFAULT; const src = opts.src || SRC_DEFAULT;
const dist = opts.dist || DIST_DEFAULT; const dist = opts.dist || DIST_DEFAULT;
const bannerUrl =
opts.bannerUrl || path.join(__dirname, SRC_DEFAULT, '_copyright');
const commentStyle = opts.commentStyle || 'ignored';
return { return {
input: [`${src}/${filename}.js`], input: `${src}/${filename}.js`,
output: { output: {
file: `${dist}/${filename}.min.js`, file: `${dist}/${filename}.min.js`,
format: 'iife', format: 'iife',
name: 'Chirpy', name: 'Chirpy',
banner,
sourcemap: !isProd sourcemap: !isProd
}, },
watch: { watch: {
@ -46,18 +51,13 @@ function build(filename, opts = {}) {
}), }),
nodeResolve(), nodeResolve(),
yaml(), yaml(),
isProd && commentStyle === 'none' && terser(), isProd && terser()
license({
banner: {
commentStyle,
content: { file: bannerUrl }
}
}),
isProd && commentStyle !== 'none' && terser()
] ]
}; };
} }
cleanup(DIST_DEFAULT, DIST_PWA);
export default [ export default [
build('commons'), build('commons'),
build('home'), build('home'),
@ -65,10 +65,6 @@ export default [
build('page'), build('page'),
build('post'), build('post'),
build('misc'), build('misc'),
build('app', { src: SRC_PWA }), build('app', { src: SRC_PWA, dist: DIST_PWA }),
build('sw', { build('sw', { src: SRC_PWA, dist: DIST_PWA })
src: SRC_PWA,
bannerUrl: path.join(__dirname, SRC_PWA, '_frontmatter'),
commentStyle: 'none'
})
]; ];

View file

@ -92,7 +92,7 @@ init_files() {
npm i && npm run build npm i && npm run build
# track the CSS/JS output # track the CSS/JS output
_sedi "/.*\/dist$/d" .gitignore _sedi "/.*\/dist$/d;/^_app$/d" .gitignore
} }
commit() { commit() {

View file

@ -17,6 +17,7 @@ CONFIG="_config.yml"
CSS_DIST="_sass/dist" CSS_DIST="_sass/dist"
JS_DIST="assets/js/dist" JS_DIST="assets/js/dist"
PWA_DIST="_app"
FILES=( FILES=(
"$GEM_SPEC" "$GEM_SPEC"
@ -111,20 +112,13 @@ prepare() {
## Build a Gem package ## Build a Gem package
build_gem() { build_gem() {
if $opt_pkg; then
BACKUP_PATH="$(mktemp -d)"
mkdir -p "$BACKUP_PATH"/css "$BACKUP_PATH"/js
[[ -d $CSS_DIST ]] && cp "$CSS_DIST"/* "$BACKUP_PATH"/css
[[ -d $JS_DIST ]] && cp "$JS_DIST"/* "$BACKUP_PATH"/js
fi
# Remove unnecessary theme settings # Remove unnecessary theme settings
sed -i -E "s/(^timezone:).*/\1/;s/(^cdn:).*/\1/;s/(^avatar:).*/\1/" $CONFIG sed -i -E "s/(^timezone:).*/\1/;s/(^cdn:).*/\1/;s/(^avatar:).*/\1/" $CONFIG
rm -f ./*.gem rm -f ./*.gem
npm run build npm run build
# add CSS/JS distribution files to gem package # add CSS/JS distribution files to gem package
git add "$CSS_DIST" "$JS_DIST" -f git add "$CSS_DIST" "$JS_DIST" "$PWA_DIST" -f
echo -e "\n> gem build $GEM_SPEC\n" echo -e "\n> gem build $GEM_SPEC\n"
gem build "$GEM_SPEC" gem build "$GEM_SPEC"
@ -132,14 +126,6 @@ build_gem() {
echo -e "\n> Resume file changes ...\n" echo -e "\n> Resume file changes ...\n"
git reset git reset
git checkout . git checkout .
if $opt_pkg; then
# restore the dist files for future development
mkdir -p "$CSS_DIST" "$JS_DIST"
cp "$BACKUP_PATH"/css/* "$CSS_DIST"
cp "$BACKUP_PATH"/js/* "$JS_DIST"
rm -rf "$BACKUP_PATH"
fi
} }
# Push the gem to RubyGems.org (using $GEM_HOST_API_KEY) # Push the gem to RubyGems.org (using $GEM_HOST_API_KEY)