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.

Sign up for GitHub

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

Jump to bottom

Update unit tests #105

Merged
merged 2 commits into from
Dec 22, 2017
Merged

Update unit tests #105

merged 2 commits into from
Dec 22, 2017

Conversation

extsoft
Copy link
Contributor

@extsoft extsoft commented Dec 22, 2017

Solves #93
Affects #83

@extsoft
Use real git project to test local config
4613c66 
Use 'git config <key>' to read default values.
Start local or global configuration from user options.

#93
@extsoft
Use 'check' instead of 'run' in all unit tests
d6af6a3
@extsoft
Copy link
Contributor Author

extsoft commented Dec 22, 2017

@rultor please merge.

@extsoft extsoft closed this Dec 22, 2017
@extsoft extsoft reopened this Dec 22, 2017
@rultor
Copy link

rultor commented Dec 22, 2017

@rultor please merge.

@extsoft OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit d6af6a3 into master Dec 22, 2017
@extsoft extsoft deleted the 93 branch December 22, 2017 16:53
@rultor
Copy link

rultor commented Dec 22, 2017

@rultor please merge.

@extsoft Done! FYI, the full log is here (took me 2min)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@extsoft @rultor