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.

Sign up for GitHub

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

Jump to bottom

Use npm to manage fomantic and only build needed components #9561

Merged
merged 14 commits into from
Jan 21, 2020
Merged

Use npm to manage fomantic and only build needed components #9561

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
ca23114
Use npm to manage fomantic
lunny Dec 31, 2019
d195d0d
Only build needed semantic components
lunny Jan 1, 2020
a56ee78
Fix make
lunny Jan 1, 2020
0ee3b2b
Don't import fonts from google sites since we have loaded
lunny Jan 1, 2020
eb3c9ea
[misc] devendor fomantic-ui and rebuild upon src or config changes only
das7pad Jan 6, 2020
123ee04
Change sort alphabetically of semantic components
lunny Jan 7, 2020
07d0b71
Fix trailing slash
lunny Jan 7, 2020
1764412
fix makefile
lunny Jan 7, 2020
627bd16
Remove dependency to gulp from package.json
lunny Jan 9, 2020
02d0f79
Fix something
lunny Jan 12, 2020
b3013b1
Simplife the makefile
lunny Jan 12, 2020
4f30548
add missed fomantic compnent
lunny Jan 20, 2020
7818e13
Merge branch 'master' into lunny/fomantic_npm
lunny Jan 20, 2020
b11bdba
Merge branch 'master' into lunny/fomantic_npm
lunny Jan 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
Simplife the makefile
  • Loading branch information
@lunny
lunny committed Jan 20, 2020
commit b3013b14709d15ae71e54f7a53ff92604dd9c3fb
14 changes: 3 additions & 11 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -480,18 +480,10 @@ $(JS_DEST): node_modules $(JS_SOURCES)
npx webpack

.PHONY: fomantic
fomantic: $(FOMANTIC_DEST_DIR)
node_modules/fomantic-ui/src: node-check node_modules

# TODO(das7pad): replace with resolve.alias when building fomantic with webpack
node_modules/fomantic-ui/src/theme.config: node_modules/fomantic-ui/src
node_modules/fomantic-ui/src/theme.config: web_src/fomantic/theme.config.less
ifeq (,$(wildcard $@))
cp $< $@
endif
fomantic: node-check $(FOMANTIC_DEST_DIR)

$(FOMANTIC_DEST_DIR): node_modules/fomantic-ui/src
$(FOMANTIC_DEST_DIR): node_modules/fomantic-ui/src/theme.config
$(FOMANTIC_DEST_DIR): node_modules semantic.json web_src/fomantic/theme.config.less
cp web_src/fomantic/theme.config.less node_modules/fomantic-ui/src/theme.config
npx gulp -f node_modules/fomantic-ui/gulpfile.js build

Copy link
Member

Choose a reason for hiding this comment

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

Maybe it is possible to have this as a non-phony target that depends on package-lock.json, so it would only run if dependencies change.

Copy link
Member

@silverwind silverwind Jan 4, 2020 •
edited

Choose a reason for hiding this comment

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

Also, I'd suggest the target to be adjusted to provide one or more output files, like public/fomantic/semantic.min.js and public/fomantic/semantic.min.css. We can still have a phony fomantic target that depends on that file to trigger it on-demand.

.PHONY: css
Expand Down