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

Minor config improvements #101

Merged
merged 2 commits into from
Jan 14, 2017
Merged

Minor config improvements #101

merged 2 commits into from
Jan 14, 2017

Conversation

falexandrou
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 94.69% when pulling c124ea2 on falexandrou:config-changes into 4f8b64a on hirakiuc:master.

@hirakiuc hirakiuc added this to the v1.4.0 milestone Dec 11, 2016
@hirakiuc
Copy link
Owner

hirakiuc commented Jan 8, 2017

Sorry, I missed checking this pull request...
I already added access_token config in #106. 🙇

Please remove access_token config and fix conflicts. 🙏

@coveralls
Copy link

Coverage Status

Coverage increased (+0.004%) to 94.709% when pulling 12f0123 on falexandrou:config-changes into 5207b29 on hirakiuc:master.

@hirakiuc
Copy link
Owner

I fixed the conflict and LGTM.

Thanks!

@hirakiuc hirakiuc merged commit 5d5d45a into hirakiuc:master Jan 14, 2017
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
enhancement
Projects
None yet
Milestone
v1.4.0
Development

Successfully merging this pull request may close these issues.
3 participants
@falexandrou @coveralls @hirakiuc