Skip to content
This repository has been archived by the owner on Apr 24, 2019. It is now read-only.

Add a modules folder for all components on a page #127

Merged
merged 1 commit into from
Dec 1, 2014

Conversation

drublic
Copy link
Member

@drublic drublic commented Nov 26, 2014

Usually you want to develop components for your website or app which
should ideally be bundled into one directory. This commit adds a
directory modules for all these modules and updates according
configuration for Karma and the build process.

Reference: #123

Please have a detailed look if this makes a) sense to you and b) works as expected.

Usually you want to develop components for your website or app which
should ideally be bundled into one directory. This commit adds a
directory `modules` for all these modules and updates according
configuration for Karma and the build process.

Reference: #123
@drublic drublic closed this Nov 27, 2014
@drublic drublic deleted the feature/module-folder branch November 27, 2014 09:33
@drublic drublic restored the feature/module-folder branch November 27, 2014 09:33
@drublic drublic reopened this Nov 27, 2014
@anselmh anselmh added this to the v2.0.0 milestone Nov 27, 2014
@drublic drublic merged commit 8ba4833 into v2.0 Dec 1, 2014
@drublic drublic deleted the feature/module-folder branch December 1, 2014 07:14
@anselmh
Copy link
Member

anselmh commented Dec 1, 2014

Thanks, works fine here. I wonder if we should allow html to be in there, too (for consistency mainly)? But to be honest I don't think people would use this approach for HTML and you'll always will have some generic html elsewhere. What do you think?

@drublic
Copy link
Member Author

drublic commented Dec 1, 2014

I think it would be very nice to have HTML in this folder. Care to create a PR?

@anselmh
Copy link
Member

anselmh commented Dec 1, 2014

I could easily do but earliest by tomorrow (don't have any time today unfortunately).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants