Skip to content

Features/incremental build#569

Merged
bmuenzenmeyer merged 15 commits intodevfrom
features/incremental-build
Dec 12, 2016
Merged

Features/incremental build#569
bmuenzenmeyer merged 15 commits intodevfrom
features/incremental-build

Conversation

@tburny
Copy link
Copy Markdown

@tburny tburny commented Nov 30, 2016

_Addresses #540 _

Summary of changes:

This is a prototype for recompiling any templates that have been modified since they have been previously built. This works by inspecting the source and output files' modification timestamps.
A pattern is marked as needs rebuild if any of its source files was modified after the modification timestamp of the output file. When the template was built and the output was written to disk, it is marked as clean to prevent building templates more than once.

Working

A pattern is recompiled iif

  • the template itself was modified
  • its data (companion json) was modified
  • it is a pseudo-pattern and the according json file was modified
  • existing unit tests are running

Not working yet

  • When using grunt, patternlab:serve will wipe the output folder. We can provide a seperate goal.

@bmuenzenmeyer
Copy link
Copy Markdown
Member

@tburny thanks for this - and all the comments. Like we discussed - I will start combing through it asap for a 2.8.0-alpha.

In the meantime, if you could attempt to fix the eslint errors that'd help a ton. npm test reports them locally too.

@tburny
Copy link
Copy Markdown
Author

tburny commented Dec 1, 2016

In the meantime, if you could attempt to fix the eslint errors that'd help a ton. npm test reports them locally too.

Sure.

Tobias Brennecke added 6 commits December 1, 2016 21:37
When changing the structure of a PatternGraph, users might still have an old version, so we need to rebuild all patterns and the whole graph.
@bmuenzenmeyer
Copy link
Copy Markdown
Member

@tburny I have started testing this today

@bmuenzenmeyer
Copy link
Copy Markdown
Member

bmuenzenmeyer commented Dec 12, 2016

@tburny appears broken if defaultPattern is not all

my test:

"defaultPattern": "start-welcome",

changing it to all caused it to build, albeit with improper order. still need to merge dev in

@bmuenzenmeyer
Copy link
Copy Markdown
Member

scratch that - merging dev worked!!!!

@bmuenzenmeyer
Copy link
Copy Markdown
Member

  • Merged dev into features/incremental-build locally
  • Using edition-node-gulp gulpfile
  • Testing with cleanPublic false,
  • Running gulp patternlab:serve

This outputs only changed patterns.

3.2.1. 🚀 >>>>>>>>>>>>>>>>>>>>>🎯 💣

image

@bmuenzenmeyer bmuenzenmeyer merged commit b0bde03 into dev Dec 12, 2016
@bmuenzenmeyer bmuenzenmeyer deleted the features/incremental-build branch December 12, 2016 21:22
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.

2 participants