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

Commit

Permalink
Auto merge of #5010 - aycabta:use-require-instead-of-autoload-for-plu…
Browse files Browse the repository at this point in the history
…gin-api, r=indirect

[workaround] Use `require` instead of `autoload` for bundler/plugin/api

Please read #4981 (comment).

> But it's elusive reproduction (and lack of test case) makes me think that the problem is a bit deeper and may recur.

I think so. This Pull Request is just *workaround*.
  • Loading branch information
homu committed Oct 3, 2016
2 parents e11639f + 4c32972 commit 8168619
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion lib/bundler/plugin.rb
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
# frozen_string_literal: true
require "bundler/plugin/api"

module Bundler
module Plugin
autoload :API, "bundler/plugin/api"
autoload :DSL, "bundler/plugin/dsl"
autoload :Index, "bundler/plugin/index"
autoload :Installer, "bundler/plugin/installer"
Expand Down
2 changes: 1 addition & 1 deletion lib/bundler/plugin/api.rb
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# frozen_string_literal: true
require "bundler/plugin/api/source"

module Bundler
# This is the interfacing class represents the API that we intend to provide
Expand All @@ -23,7 +24,6 @@ module Bundler
# and hooks).
module Plugin
class API
autoload :Source, "bundler/plugin/api/source"
# The plugins should declare that they handle a command through this helper.
#
# @param [String] command being handled by them
Expand Down

0 comments on commit 8168619

Please sign in to comment.