How to Contribute to Open Source - zenspider.com€¦ · How to Contribute to Open Source...

Madison Ruby 2012, Madison, WI

Ryan Davis, Seattle.rbHow to Contribute to Open Source

How to Contribute to Open Source

Extensibility from Simplicity

Ryan Davis, Seattle.rb

Madison!

Don’t touch“Montecello”

Lake Monona

Lake mono-Na

As in: one “na”

Sha-na-naBa-na-na

Dodeca-na

na na na nana na na nana na na na

http://en.wikipedia.org/wiki/Na_Na_Na_(Na_Na_Na_Na_Na_Na_Na_Na_Na)

Double Dodeca-na

na na na na na nana na na na na nana na na na na nana na na na na na

I Lied

Aaron Patterson

talked about how to get

started contributing

to OSS.

Giving BackA practical guide to OSS contributions

I won't be talking about getting started.

He did a better job than I could.

The video will be up on confreaks.

…Eventually.

http://juliepagano.com/blog/2012/08/05/steel-city-ruby-conf-2012-sketchnotes/

Holy crap, Communicate!

As a Sidenote…

Aaron sucks and you never want to work

with him.

It is embarrassing to go out to lunch with

Embarrassing to work in public with

All he talks about is his fugly cat.

Who ran into a brick wall as a

kitten.

…and his sausage.

A Typical Day…

Why are you meowing?

Thinking about cats.

Why are you meowing Journey's

“Don't Stop Believing”?

Thinking about cats...

singing.

Thinking about cats...

singing.

Grrr…

That said...

He's brilliant and working with him is

fun and rewarding.

My Talk

Later stages of open source contribution,both the developer and contributor standpoints.

What are contributors?

Consumers

Developers

Consumers

Developers

Contributors

ProprietarySoftware

OSSSoftware

What are contributions?

Feature Requests

Documentation

Feedback

Ye OldeOpen Source

Started with smoke signals…

http://efuse.com/Grow/www-smoke-signals-rob-colvin-artville.jpg

Check out project CVS or SVN

Work your changes.

Submit diffs back to developer.

Hopefully there was sourceforge.

Most of the burden was on the user to

get it right.

Nowadays…

Being a contributor is easier than ever!

Now we use git.(or hg or darcs or ...)

On github…

Pull Requestsinstead of patch sets.

You don't even need a local checkout

anymore!

Forked projects and Pull requests have

scaled up dramatically.

This is good, but it also means that the project

developers are massively outnumbered.

http://www.imdb.com/title/tt0098546/

http://www.jwz.org/blog/2008/03/you-get-to-drink-from-the-firehose/

Most of the burden goes back to the

developer.

When should someone try to

contribute?

Early and often!

How to handle a contribution?

Developer has roughly 3 options:

Accept As-Is

ComplexityDramaEffort

Reject

Rewrite/Distill

Rewrite

Distillation Example

minitest has a flexible but “simple” runner

“Simple”MiniTest::Unit.autorun MiniTest::Unit.new.run ARGV self.class.runner._run args send plugin run_tests _run_anything :test _run_suites suites, type _run_suite suite, type tc.run self before_setup setup after_setup run_test __name__ __send__ __name__ before_teardown teardown after_teardown

Allows for:

Running tests.

Running n×m benchmarks

w/ curve fitting.

It has hooks for various phases of a

test run.

Aaron helped me generalize the hook

system.

To meet his needs for testing rails with

mocks.

He has the benefit of sitting next to me.

Most otherdevelopers don’t.

Mocha, turn,CI systems...

Number of bugs filed?

copy/paste/munge./lib/mocha/integration/mini_test/assertion_counter.rb./lib/mocha/integration/mini_test/exception_translation.rb./lib/mocha/integration/mini_test/version_13.rb./lib/mocha/integration/mini_test/version_140.rb./lib/mocha/integration/mini_test/version_141.rb./lib/mocha/integration/mini_test/version_142_to_172.rb./lib/mocha/integration/mini_test/version_200.rb./lib/mocha/integration/mini_test/version_201_to_222.rb./lib/mocha/integration/mini_test/version_2110_to_2111.rb./lib/mocha/integration/mini_test/version_2112_to_320.rb./lib/mocha/integration/mini_test/version_230_to_2101.rb./lib/mocha/integration/mini_test/version_330.rb./lib/mocha/integration/mini_test.rb

complexity = #frameworks × #releases

Every time minitest’s internals change, they (can) break.

Untenable

Happy Example

IRC Dialog

co-worker: I was trying to make something similar to this: http://caldersphere.rubyforge.org/ci_reporter/...zenspider: unfamiliar... seczenspider: oh godzenspider: that integration is crazy

Monkeypatch of 224 lines! ARGH!

He submitted a pull request.

Trying to make his life easier.

Patch was the wrong way to go.

There's already a huge difference.

It got the ball rolling.

He was communicating.

This gave us the chance to reach

some clarity.

Small change to minitest### Record the result of a single run. Makes it very easy to gather# information. Eg:## class StatisticsRecorder < MiniTest::Unit# def record suite, method, assertions, time, error# # ... record the results somewhere ...# end# end## MiniTest::Unit.runner = StatisticsRecorder.new

def record suite, method, assertions, time, errorend

# in _run_suite:record suite, method, inst._assertions, time, last_error

Massively Simplified His Codediff --git a/.autotest b/.autotestindex 5cd7c43..f4f90a1 100644--- a/.autotest+++ b/.autotest@@ -1,26 +1,11 @@ # -*- ruby -*- require 'autotest/restart'+require 'autotest/isolate' Autotest.add_hook :initialize do |at| at.add_exception 'test/reports'-# at.extra_files << "../some/external/dependency.rb"-#-# at.libs << ":../some/external"-#-# at.add_exception 'vendor'-#-# at.add_mapping(/dependency.rb/) do |f, _|-# at.files_matching(/test_.*rb$/)-# end-#-# %w(TestA TestB).each do |klass|-# at.extra_class_map[klass] = "test/test_misc.rb"-# end false end -# Autotest.add_hook :run_command do |at|-# system "rake build"-# end # vim: syntax=rubydiff --git a/README.txt b/README.txtindex 92c840f..5898fb2 100644--- a/README.txt+++ b/README.txt@@ -5,11 +5,12 @@ == DESCRIPTION: CI reporter plugin for MiniTest+ This gem was made possible by ATT Interactive. == FEATURES/PROBLEMS: -* Monkeypatches MiniTest::Unit#_run_suite so I have to fix the version of MiniTest I support.+* Uses MiniTest::Unit#record which is only available since version 2.12.0 == SYNOPSIS: diff --git a/Rakefile b/Rakefileindex ff07b81..5e1ae7e 100644--- a/Rakefile+++ b/Rakefile@@ -10,7 +10,7 @@ Hoe.spec 'minitest-ci' do self.testlib = :none - extra_deps << ['minitest', '2.6.0']+ extra_deps << ['minitest', '~> 2.12.0'] extra_dev_deps << ['nokogiri', '~> 1.5.0'] extra_dev_deps << ['rdoc', '>= 2.4.2'] enddiff --git a/lib/minitest/ci.rb b/lib/minitest/ci.rbindex 6c1c5ba..02b1bc7 100644--- a/lib/minitest/ci.rb+++ b/lib/minitest/ci.rb@@ -1,12 +1,7 @@-begin- gem 'minitest', '2.6.0'- require "minitest/unit"-rescue Gem::LoadError => e- msg = [- 'minitest version 2.6.0 required',- 'try: gem install minitest -v 2.6.0'- ]- raise e.exception msg.join("\n")

+require 'minitest/unit'+#unless MiniTest::Unit::VERSION >= '2.12.0'+unless MiniTest::Unit.public_method_defined? :record+ abort 'you are running an unsupported version of MiniTest. try upgrading.' end require 'fileutils'@@ -14,44 +9,20 @@ module MiniTest module Ci- ###- # copied- file = if RUBY_VERSION =~ /^1\.9/ then # bt's expanded, but __FILE__ isn't :(- File.expand_path __FILE__- elsif __FILE__ =~ /^[^\.]/ then # assume both relative- require 'pathname'- pwd = Pathname.new Dir.pwd- pn = Pathname.new File.expand_path(__FILE__)- pn = File.join(".", pn.relative_path_from(pwd)) unless pn.relative?- pn.to_s- else # assume both are expanded- __FILE__- end- # end of copy- ###- CI_MINI_DIR = File.dirname(File.dirname(file)) VERSION = '1.0.4' - @test_dir = nil #'test/reports'- @error = nil+ @report_dir = nil @suites = Hash.new {|h,k| h[k] = []} class << self- attr_accessor :test_dir+ # Accessor method to change the report dir if you don't like the+ # defaults.+ attr_accessor :report_dir end - def add_error e- @error = e- end-- def push suite, method, num, time- a = [method, time, num]- if @error- a << @error- @error = nil- end- @suites[suite] << a+ def push suite, method, num_assertions, time, error+ @suites[suite] << [method, num_assertions, time, error] end def finish io@@ -60,7 +31,7 @@ def finish io clean - Dir.chdir @test_dir do+ Dir.chdir report_dir do @suites.each do |name, suite| generate_suite name, suite end@@ -70,31 +41,17 @@ def finish io private def clean- FileUtils.rm_rf @test_dir- FileUtils.mkdir_p @test_dir+ FileUtils.rm_rf report_dir+ FileUtils.mkdir_p report_dir

end def escape o CGI.escapeHTML(o.to_s) end - # use original as well as filtering this file.- def filter_backtrace bt- bt = MiniTest::filter_backtrace bt- orig_mini_dir = MiniTest::MINI_DIR-- orig_verbose, $VERBOSE = $VERBOSE, nil-- MiniTest.const_set :MINI_DIR, CI_MINI_DIR- MiniTest::filter_backtrace bt- ensure- MiniTest.const_set :MINI_DIR, orig_mini_dir- $VERBOSE = orig_verbose- end- def generate_suite name, suite total_time, assertions, errors, failures, skips = 0, 0, 0, 0, 0- suite.each do |_, t, a, e|+ suite.each do |_, a, t, e| total_time += t assertions += a case e@@ -111,12 +68,12 @@ def generate_suite name, suite f.puts '<?xml version="1.0" encoding="UTF-8"?>' f.puts "<testsuite time='#{"%6f" % total_time}' skipped='#{skips}' failures='#{failures}' errors='#{errors}' name='#{name}' assertions='#{assertions}' tests='#{suite.count}'>" - suite.each do |method, time, asserts, error|+ suite.each do |method, asserts, time, error| f.puts " <testcase time='#{"%6f" % time}' name='#{method}' assertions='#{asserts}'>" if error- bt = filter_backtrace(error.backtrace).join "\n"+ bt = MiniTest::filter_backtrace error.backtrace f.write " <#{type error} type='#{escape error.class}' message=#{escape( error.message ).inspect}>"- f.puts escape bt+ f.puts escape bt.join "\n" f.puts " </#{type error}>" end f.puts " </testcase>"@@ -138,55 +95,10 @@ def type e extend self end - class Unit- # pull this out of _run_suite to make it easier to grab stuff- def _run_suite_method suite, method- inst = suite.new method- inst._assertions = 0-- print "#{suite}##{method} = " if @verbose-- @start_time = Time.now- result = inst.run self- time = Time.now - @start_time-- print "%.2f s = " % time if @verbose- print result- puts if @verbose-- [inst._assertions, time]- end-- # copied out of MiniTest::Unit- def _run_suite suite, type- header = "#{type}_suite_header"- puts send(header, suite) if respond_to? header-- filter = options[:filter] || '/./'- filter = Regexp.new $1 if filter =~ /\/(.*)\//-

- assertions = suite.send("#{type}_methods").grep(filter).map { |method|- a_count, _ = _run_suite_method suite, method-- a_count- }-- return assertions.size, assertions.inject(0) { |sum, n| sum + n }- endend- class CiUnit < Unit - def _run_suite_method suite, method- res = super-- MiniTest::Ci.push suite, method, *res-- res- end-- def puke klass, meth, e- MiniTest::Ci.add_error e+ def record suite, method, assertions, time, error+ MiniTest::Ci.push suite, method, assertions, time, error super end @@ -199,5 +111,5 @@ def status io = self.output end # set defaults-MiniTest::Ci.test_dir = 'test/reports'+MiniTest::Ci.report_dir = 'test/reports' MiniTest::Unit.runner = MiniTest::CiUnit.newdiff --git a/test/test_minitest_ci.rb b/test/test_minitest_ci.rbindex 1b204ec..6cfffce 100644--- a/test/test_minitest_ci.rb+++ b/test/test_minitest_ci.rb@@ -49,14 +49,14 @@ class TestMinitest::TestCi < MiniTest::Unit::TestCase ensure MiniTest::Unit.output = old_out end+ MiniTest::Unit.runner = nil # reset def self.output @output end def setup- file = "#{MiniTest::Ci.test_dir}/TEST-MockTestSuite.xml"- assert File.exists?( file ), 'expected xml file to exists'+ file = "#{MiniTest::Ci.report_dir}/TEST-MockTestSuite.xml" @file = File.read file @doc = Nokogiri.parse @file @doc = @doc.at_xpath('/testsuite')@@ -113,12 +113,8 @@ def test_output assert_match( /generating ci files/, self.class.output.read ) end - def test_filter_backtrace- bt = begin; raise; rescue => e; e.backtrace; end-- bt = MiniTest::Ci.send :filter_backtrace, bt-- assert_equal 1, bt.size- assert_match( /#{__FILE__}/, bt.first )+ def test_filtering_backtraces+ error = @doc.at_xpath('/testsuite/testcase[@name="test_raise_error"]')+ refute_match /lib\/minitest/, error.inner_text end end-- 1.7.10

His code now looks like:class CiUnit < MiniTest::Unit after_tests do MiniTest::Ci.finish self.output end

def record suite, method, assertions, time, error MiniTest::Ci.push suite, method, assertions, time, error super endend

MiniTest::Unit.runner = MiniTest::CiUnit.new

Conclusion

Communicate Communicate Communicate Communicate

!ank You

How to Contribute to Open Source - zenspider.com€¦ · How to Contribute to Open Source...

Documents

Improving Coverage Analysis - zenspider.com · 2020-05-16 · Improving Coverage Analysis RubyConf 2016, Cincinnati, OH Ryan Davis, Seattle.rb @the_zenspider Ryan Davis Coding professionally

Contribute to Symfony

Unit I Minitest - mhpracticeplus.com · Unit I Minitest 32 Questions 30 Minutes ... (12900 N) attached to a 30 × 106 cubic feet (8.5 × 108 L) helium-ﬁlled weather bal-loon. The

Contribute Training

MT 7200 7400 FH 03g Engl - Tension, Torque, Force ... Manual.pdf · Designed for non-destructive thickness measurement, the portable MiniTest 7200 / 7400 FH measuring systems are

Jab 2012-contribute

Adobe Contribute CS5 What's New - a248.e.akamai.neta248.e.akamai.net/.../images/pdf/adobe_contribute_cs5_whatsnew.pdf · Contribute CS5 . What’s New. Adobe Contribute CS5 is also

Openobject contribute

Minitest 6: Test Feistier! Ryan Davis, Seattle.rb Minitest 6 · Presentation Sammich 7. Minitest 6: Test Feistier! RailsConf 2018, Pittsburgh, PA Ryan Davis, Seattle.rb

MiniTest 3100 - 4100...MiniTest 3100 - 4100 Technical Manual and Operating Instructions Advancing with Technology ElektroPhysik

National 5 Chemistry Unit 1 Chemical changes and structures...1 – Rates of Reaction 1 Minitest 7 2 – Elements & Atomic Structure 10 Minitest 14 3 - Bonding Related to Properties

Standing on the Shoulders of Giants - zenspider.com · Standing on the Shoulders of Giants MWRC 2015, SLC, UT Ryan Davis, Seattle.rb @the_zenspiderx "we [the Moderns] are like dwarves

Let's Contribute

Let’s Wr e an Interpreter! - zenspider.com · 2020-05-16 · GoGaRuCo 2013, San Francisco, CA Let’s Write an Interpreter! Ryan Davis, Seattle.rb Setting Expectations • 168 Slides

Trolls of 2013 - zenspider.com

Bedienungsanleitung MiniTest 7000195.159.137.154/pub/Bruksanvisninger-Delelister/07/ElektroPhysik... · 6.2.3.6 Calibration according to SSPC-PA2 ... 96 15. Annexe ... SSPC, “Swedish”,

What can optical spectroscopy contribute to understanding ... · What can optical spectroscopy contribute to understanding protein dynamics? ... optical spectroscopy contribute to

Cumulative Minitest - mhpracticeplus.com€¦ · MCAT-3200184 book November 13, 2015 14:48 MHID: 1-25-958837-8 ISBN: 1-25-958837-2 384 Cumulative Minitest Directions: Choose the best

Coating Thickness Gauges MiniTest 720, 730, 740 · Elektro Physik Technical Manual / Operating Instructions Coating Thickness Gauges MiniTest 720, 730, 740

Contribute Quickstart