Skip to content

Make knapsack backwards compatible with earlier version of minitest#26

Merged
ArturT merged 4 commits intoKnapsackPro:masterfrom
dkniffin:master
Oct 27, 2015
Merged

Make knapsack backwards compatible with earlier version of minitest#26
ArturT merged 4 commits intoKnapsackPro:masterfrom
dkniffin:master

Conversation

@dkniffin
Copy link
Copy Markdown
Contributor

No description provided.

after_run was implemented in a newer version of minitest
- Added support for earlier versions of minitest that use after_tests instead of after_run
- Slightly better way of finding the file containing the test
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.

You experienced problem with this in case when you had helper methods without test_ in name?

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.

Yea, I forget the issue I ran into, but this fixed it :/

ArturT added a commit that referenced this pull request Oct 27, 2015
Make knapsack backwards compatible with earlier version of minitest
@ArturT ArturT merged commit 5d7e4dd into KnapsackPro:master Oct 27, 2015
@ArturT
Copy link
Copy Markdown
Member

ArturT commented Oct 27, 2015

@dkniffin Thanks for awesome work!

ArturT added a commit to KnapsackPro/knapsack_pro-ruby that referenced this pull request Oct 27, 2015
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