Skip to content

Add support for Spinach#41

Merged
ArturT merged 4 commits intoKnapsackPro:masterfrom
ayufan:master
Jun 3, 2016
Merged

Add support for Spinach#41
ArturT merged 4 commits intoKnapsackPro:masterfrom
ayufan:master

Conversation

@ayufan
Copy link
Copy Markdown
Contributor

@ayufan ayufan commented Jun 3, 2016

This adds support for Spinach: https://github.com/codegram/spinach

This MR covers:

  • a new SpinachAdapter,
  • a rspec tests for SpinachAdapter,
  • a spinach_examples to run test on Travis-CI,
  • updates README.md with description how to use it.

@ArturT
Copy link
Copy Markdown
Member

ArturT commented Jun 3, 2016

@ayufan Great work. Thank you!

@ArturT ArturT merged commit 839184b into KnapsackPro:master Jun 3, 2016
@ayufan
Copy link
Copy Markdown
Contributor Author

ayufan commented Jun 3, 2016

@ArturT Will you release a new version of gem? :)

@ArturT
Copy link
Copy Markdown
Member

ArturT commented Jun 3, 2016

@ayufan Yes. I'm on it. I need to do a few small changes and I will let you know.

@ArturT
Copy link
Copy Markdown
Member

ArturT commented Jun 3, 2016

@ayufan I've just released knapsack 1.11.0. :)

@@ -0,0 +1,42 @@
require 'spinach'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@ayufan I found out this causes problems when someone don't use spinach. I did fix in knapsack 1.11.1
https://github.com/ArturT/knapsack/blob/master/CHANGELOG.md#1111

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks :) I appreciate it 👍

@ArturT
Copy link
Copy Markdown
Member

ArturT commented Jun 3, 2016

@ayufan I added Spinach support also to knapsack_pro gem. Maybe you would like to try it, it's free? I'd love to hear your feedback. http://knapsackpro.com You have a big project so it might be interesting how the knapsack_pro will work for you.

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.

2 participants