Gemify! #48

Merged
merged 4 commits into from Feb 22, 2013

Conversation

Projects
None yet
3 participants
Contributor

xiongchiamiov commented Feb 16, 2013

Write a simple gemspec, move things into their appropriate directories,
and add a little Rake task to make it easier to generate a gem, install
it, publish it, etc.

Fixes #43.

xiongchiamiov added some commits Feb 16, 2013

@xiongchiamiov xiongchiamiov Delete Gemfile.lock.
Per Yehuda Katz's advice[0], ditch the specific versions of gems specified
in the Gemfile.lock so that we are forced to get the dependencies
specified correctly in our Gemfile (or future gemspec).

[0]: http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/
1bb360c
@xiongchiamiov xiongchiamiov Gemify!
Write a simple gemspec, move things into their appropriate directories,
and add a little Rake task to make it easier to generate a gem, install
it, publish it, etc.

Fixes #43.
84c9127
Contributor

xiongchiamiov commented Feb 16, 2013

I haven't published it yet. It's installed on Cominor at ~bsomers/build/bin/{feature|hotfix}.

@xiongchiamiov xiongchiamiov Update installation instructions.
Also, fix incorrect ordering of example `ln` command.
1b35f97
Owner

danielbeardsley commented Feb 16, 2013

Exciting!

@timothyasp timothyasp commented on the diff Feb 22, 2013

lib/github.rb
@@ -1,7 +1,6 @@
require "rubygems"
require "bundler/setup"
require 'octokit'
-require 'readline'
@xiongchiamiov

xiongchiamiov Feb 22, 2013

Contributor

I didn't pay enough attention to addd628; I thought that removed all uses of readline, but it only removed one. I'll fix it.

@xiongchiamiov xiongchiamiov Re-add readline import.
I mistakenly removed it in 84c9127 because I thought it had been
replaced entirely by highline in addd628; this is not true.

Readline is part of the standard library, so there's no need to include
it in the dependency list.
70f9faf

@xiongchiamiov xiongchiamiov added a commit that referenced this pull request Feb 22, 2013

@xiongchiamiov xiongchiamiov Merge gemify (#48) into master
Gemify!

Write a simple gemspec, move things into their appropriate directories,
and add a little Rake task to make it easier to generate a gem, install
it, publish it, etc.

Fixes #43.
0333e8a

@xiongchiamiov xiongchiamiov merged commit 70f9faf into master Feb 22, 2013

Contributor

xiongchiamiov commented Feb 22, 2013

Merged and published to RubyGems.

Owner

danielbeardsley commented Feb 24, 2013

Sweet!

xiongchiamiov added the r-All label Feb 27, 2014

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