forks/forgejo
1
0
Fork 0
Code Releases Activity

Fix webpack build warnings (#13752)

Browse source 
Turns out some of the dependencies make use of optional dependencies
like the vue-template-compiler. Install them again and install
fomantic-ui only when it's being build to fix webpack warnings.
This commit is contained in:
silverwind 2020-11-30 00:16:04 +01:00 committed by GitHub
parent e81edc02b9
commit 63aeb0a859
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 543 additions and 4406 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Makefile
View file

@ -623,7 +623,7 @@ docs:
cd docs; make trans-copy clean build-offline; cd docs; make trans-copy clean build-offline;
node_modules: package-lock.json node_modules: package-lock.json
npm install --no-save --no-optional npm install --no-save
@touch node_modules @touch node_modules
.PHONY: npm-update .PHONY: npm-update
@ -638,7 +638,7 @@ fomantic: $(FOMANTIC_DEST)
$(FOMANTIC_DEST): $(FOMANTIC_CONFIGS) | node_modules $(FOMANTIC_DEST): $(FOMANTIC_CONFIGS) | node_modules
@if [ ! -d node_modules/fomantic-ui ]; then \ @if [ ! -d node_modules/fomantic-ui ]; then \
npm install --no-save; \ npm install --no-save --no-package-lock fomantic-ui@2.8.7; \
fi fi
rm -rf $(FOMANTIC_DEST_DIR) rm -rf $(FOMANTIC_DEST_DIR)
cp -f web_src/fomantic/theme.config.less node_modules/fomantic-ui/src/theme.config cp -f web_src/fomantic/theme.config.less node_modules/fomantic-ui/src/theme.config

4942
package-lock.json generated
View file

File diff suppressed because it is too large Load diff

3
package.json
View file

@ -52,9 +52,6 @@
"worker-loader": "3.0.5", "worker-loader": "3.0.5",
"wrap-ansi": "7.0.0" "wrap-ansi": "7.0.0"
}, },
"optionalDependencies": {
"fomantic-ui": "2.8.7"
},
"devDependencies": { "devDependencies": {
"eslint": "7.14.0", "eslint": "7.14.0",
"eslint-plugin-html": "6.1.1", "eslint-plugin-html": "6.1.1",