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

just Keyword tests from PR441. Pass on Fedora #474

Merged
merged 2 commits into from
Sep 30, 2015
Merged

Conversation

tmylk
Copy link
Contributor

@tmylk tmylk commented Sep 30, 2015

PR #441 is quite large so splitting it into two chunks: tests and the changes. This is the test part.

tmylk added a commit that referenced this pull request Sep 30, 2015
@tmylk tmylk merged commit 962c884 into develop Sep 30, 2015
@piskvorky
Copy link
Owner

@tmylk technical note: we always want the PR description to be filled in (not "No description provided"). The description should explain overall "why" / rationale / caveats / tradeoffs.

After merging a PR, delete the branch to clean up.

Thanks for fixing :)

@tmylk tmylk deleted the pr_441_just_tests branch October 5, 2015 19:27
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