Warnings with Bundler 1.9.1 from Molinillo

[zzak]

Not sure if these were fixed in master yet.. or who to report it to..

/home/travis/.rvm/gems/ruby-2.2.0/gems/bundler-1.9.1/lib/bundler/vendor/molinillo/lib/molinillo/dependency_graph.rb:64: warning: possible reference to past scope - vertices
/home/travis/.rvm/gems/ruby-2.2.0/gems/bundler-1.9.1/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb:106: warning: private attribute?
/home/travis/.rvm/gems/ruby-2.2.0/gems/bundler-1.9.1/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb:109: warning: private attribute?
/home/travis/.rvm/gems/ruby-2.2.0/gems/bundler-1.9.1/lib/bundler/vendor/molinillo/lib/molinillo/resolution.rb:112: warning: private attribute?

[segiddins]

Those warnings disagree with the Molinillo style guide.

[JuanitoFatas]

Possible related: https://bugs.ruby-lang.org/issues/10967

[zzak]

I'm not familiar with Molinillo, or their style guide.. but it would be nice to fix these somehow.

Should I report this upstream?

[segiddins]

@zzak I am Molinillo, and my/the CocoaPods style guide is definitely in favor of using private attributes over direct ivar access.

[indirect]

We generally try to keep the Bundler codebase from containing any warnings, but the private attribute one seems especially weird/contested to me. :/

[segiddins]

I just fixed the first warning on Molinillo master, but I'm not changing the use of private attributes -- that's a silly warning imho.

[segiddins]

This is fixed in 1-9-stable.

[indirect]

This will be part of 1.9.2, released shortly.