Skip to content
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

Justin extensions #8

Closed
wants to merge 13 commits into from
Closed

Conversation

justin808
Copy link

see commit history

Second comparison of this method in awesome_print/formatter.rb causes
the crash.

Without this fix, pry + awesome_print cannot be used, as even a simple
case such as

[1, 2, 3].to_scale

will crash

    # Catch all method to format an arbitrary object.
    #------------------------------------------------------------------------------
    def awesome_self(object, type)
      if @options[:raw] && object.instance_variables.any?
        awesome_object(object)
      elsif object == ENV
        awesome_hash(object.to_hash)
      else
        colorize(object.inspect.to_s, type)
      end
    end
I bumped the version to 1.4.1.

* master:
  Updated README
  Fixed the implementation error of throw an error on comparing vector with another object, when only false should be sended
  Checking awesome_print error
  Updated license to BSD-3 (I could change it later). Added stable documentation
  Updated references
  Changed Manifest, dependences and README.rd
  Deleted bin. Added version file
  Updating to 1.4.0
  Raising in equality precludes working with AwesomePrint

Conflicts:
	.gitignore
	Gemfile
	Gemfile.lock
	lib/statsample/version.rb
This is the computation that fails
def t_r(r,size)
  r * Math::sqrt(((size)-2).to_f / (1 - r**2))
end
@v0dro
Copy link
Member

v0dro commented Aug 5, 2015

This one too #9

@agarie
Copy link
Member

agarie commented Aug 6, 2015

Thanks for pointing that out.

@agarie agarie closed this Aug 6, 2015
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.

3 participants