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.

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

Speed up loading of devtools #830

Merged
merged 3 commits into from Jun 5, 2015

Conversation

Projects
None yet
2 participants
@krlmlr
Member

krlmlr commented Jun 1, 2015

Avoid loading methods package and other heavy dependencies by default.

Loading time: Currently about 1 second, with this pull request: 0.2 seconds.

Git functionality (head method) and devtools:::r_release() still work correctly from Rscript.

krlmlr pushed a commit to krlmlr/devtools that referenced this pull request Jun 4, 2015

Kirill Müller

hadley added a commit that referenced this pull request Jun 5, 2015

Merge pull request #830 from krlmlr/fast-load
Speed up loading of devtools

@hadley hadley merged commit 98be9c3 into r-lib:master Jun 5, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls First build on master at 33.77%
Details
@hadley

This comment has been minimized.

Member

hadley commented Jun 5, 2015

Thanks! The timings were a bit different for me - 0.4s before, and 0.07 after.

@krlmlr krlmlr deleted the krlmlr:fast-load branch Jun 5, 2015

@krlmlr krlmlr referenced this pull request Jun 5, 2015

Merged

work around bogus NOTE #835

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