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

More testing around the globifier with new properties #1092

Merged
merged 3 commits into from
Nov 10, 2014
Merged

Conversation

ianoc
Copy link
Collaborator

@ianoc ianoc commented Nov 9, 2014

Comments on approach/refactors etc....

With this the errors from before get pretty quickly hit in the properties tests. No new bugs found in the making of these tests..

@ianoc ianoc changed the title WIP --> More testing around the globifier with new properties More testing around the globifier with new properties Nov 10, 2014
@johnynek
Copy link
Collaborator

closes #1091

Looks very thorough. Glad it did not find any more bugs.

johnynek added a commit that referenced this pull request Nov 10, 2014
More testing around the globifier with new properties
@johnynek johnynek merged commit c657464 into develop Nov 10, 2014
@johnynek johnynek deleted the globifierBug2 branch November 10, 2014 17:25
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

2 participants