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

Support nesting of Bundler block DSL #84

Closed
wants to merge 1 commit into from

Conversation

mshibuya
Copy link
Contributor

This pull request enables nesting of Bundler block DSL(i.e. group, platforms, git, path), which gives ability to express fairly complex dependency in Gemfile.

Any suggestions will be greatly appreciated!


def to_s
[source_entry,
ruby_version_entry,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Align the elements of an array literal if they span more than one line.

@mshibuya
Copy link
Contributor Author

Do I have to convert all of existing single-quoted requires to double-quoted?
There's many of them..

@sikachu
Copy link
Contributor

sikachu commented Jan 19, 2015

Thats fine. Please ignore Hound's comment on the quoting. I'll take a look at this this Friday.

Thank you for your patch.

@sikachu sikachu added the next-up label Feb 6, 2015
@sikachu sikachu added this to the v1.1.0 milestone Feb 6, 2015
@sikachu
Copy link
Contributor

sikachu commented Feb 6, 2015

I'm tagging this for 1.1.0.

@mshibuya
Copy link
Contributor Author

mshibuya commented Feb 7, 2015

Thanks, rebased to the master.

@mshibuya
Copy link
Contributor Author

mshibuya commented Mar 5, 2015

Did further rebasing. 😃

@sikachu sikachu modified the milestones: v1.1.0, v2.0.0 Mar 9, 2015
@sikachu
Copy link
Contributor

sikachu commented Mar 20, 2015

@mshibuya Please don't worry about further rebasing. I'll pull down and fixing the conflict very soon.

@sikachu
Copy link
Contributor

sikachu commented Apr 1, 2015

I've rebased the patch and made some fix to make sure that it works on Ruby 1.8.7 (as I added that a bit earlier) and merged that as 617e583.

I really like how the code looks now. It's so much cleaner.

Thank you so much for your patch!

@mshibuya
Copy link
Contributor Author

mshibuya commented Apr 1, 2015

How wonderful, thank you for your hard work!

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.

None yet

3 participants