-
Notifications
You must be signed in to change notification settings - Fork 21.6k
Commit
…nerated application. Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
module Rails | ||
class Git < Scm | ||
def self.clone(repos, branch=nil) | ||
`git clone #{repos}` | ||
|
||
if branch | ||
`cd #{repos.split('/').last}/` | ||
`git checkout #{branch}` | ||
end | ||
end | ||
|
||
def self.run(command) | ||
`git #{command}` | ||
end | ||
end | ||
end |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
module Rails | ||
class Scm | ||
private | ||
def self.hash_to_parameters(hash) | ||
hash.collect { |key, value| "--#{key} #{(value.kind_of?(String) ? value : "")}"}.join(" ") | ||
end | ||
end | ||
end |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
module Rails | ||
class Svn < Scm | ||
def self.checkout(repos, branch = nil) | ||
`svn checkout #{repos}/#{branch || "trunk"}` | ||
end | ||
end | ||
end |
8 comments
on commit e8cc4b1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh wow.
I didn’t realise you could comment on individual lines.
Massiveness.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, this is super cool, nice work j-mac
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vendor("borrowed.rb", <<CODE
..snip..
+ CODE
Missing right-paren?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oopsie.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah! great to see RG go into core!
It feels like the old days when I’ve run the rails command for the first time ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is pretty cool. I wonder why RG was chosen over Suprails?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@samgranieri: Jeremy reached out and suggested getting it included. We’ll be happy to take any patches to implement cool functionality in suprails but not in rails.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@samgranieri Thanks for the verbal support for Suprails, but Jeremy did the more intelligent thing and got it included into rails. I didn't try to do that with Suprails because I thought it would never get pulled into core. oops. =)
BLOAT WTF BBQ