Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Allow disabling plugins for faster preview #639

Merged
merged 1 commit into from Nov 9, 2014

Conversation

Projects
None yet
2 participants
Contributor

saivann commented Nov 8, 2014

https://github.com/bitcoin/bitcoin.org/tree/partialbuild#partial-build-for-faster-preview

This can allow contributors to significantly reduce time spent on waiting for building the website with the increasing features and languages supported.

I'll do a little more testing but so far these changes seem pretty straightforward and successfully reduce build time from ~4 minutes to ~17 seconds.

Contributor

harding commented Nov 8, 2014

@saivann this is super-duper-awesome! The long compile has been a huge drag. Massive thanks for this!

I've tested this and I get identical diff output[*] in the _site directory from bitcoin/master HEAD and this branch, so there are no unwanted changes. I also tested it with several different disabled combinations, and I got exactly what I expected each time. LGTM.

[*] Exception: the README.md is different, as expected.

Contributor

saivann commented Nov 8, 2014

@harding You should probably blame me for waiting for so long instead I guess, given how simple these changes turned out to be :) Thanks for your careful testing!

In the absence of critical feedback, this pull request will be merged on November 9th.

saivann added a commit that referenced this pull request Nov 9, 2014

Merge pull request #639 from bitcoin/partialbuild
Allow disabling plugins for faster preview

@saivann saivann merged commit 973355d into master Nov 9, 2014

@saivann saivann deleted the partialbuild branch Nov 9, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment