Skip to content

Conversation

jessedijkstra
Copy link
Contributor

Instead of compiling the stylesheets into view by default it's now an option. This decreases the initial and recompile compile time of views when developing the application and saving CSS files:

Compiling lib/myapp_web/views/course/index/view.ex (it's taking more than 10s)
Compiling lib/myapp_web/controllers/registration_controller.ex (it's taking more than 10s)
Compiling lib/myapp_web/cells/badge/view.ex (it's taking more than 10s)
Compiling lib/myapp_web/cells/departments/employee_list/view.ex (it's taking more than 10s)
Compiling lib/myapp_web/cells/navigation/main_desktop/view.ex (it's taking more than 10s)
Compiling lib/myapp_web/views/user/edit/view.ex (it's taking more than 10s)
Compiling lib/myapp_web/views/position/show/view.ex (it's taking more than 10s)

@codecov-io
Copy link

codecov-io commented Oct 12, 2017

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #5   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      2    +1     
  Lines          19     24    +5     
=====================================
+ Hits           19     24    +5
Impacted Files Coverage Δ
lib/ex_css_modules/view.ex 100% <100%> (ø)
lib/ex_css_modules/ex_css_modules.ex 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58f8e08...0e21aaa. Read the comment docs.

@jessedijkstra jessedijkstra merged commit 5a5ccde into master Oct 13, 2017
@jessedijkstra jessedijkstra deleted the jesse/add-option-to-embed-stylesheet branch October 13, 2017 08:14
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.

3 participants