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

#20 #21

Merged
merged 3 commits into from
Apr 1, 2015
Merged

#20 #21

merged 3 commits into from
Apr 1, 2015

Conversation

maurezen
Copy link
Contributor

Removed obsolete declaration

@dmarkov
Copy link

dmarkov commented Mar 31, 2015

Thanks, I will ask someone to do a code review here, thanks for your pull request

@dmarkov
Copy link

dmarkov commented Mar 31, 2015

@longtimeago it's yours, please review

@longtimeago
Copy link

@rultor merge please

@rultor
Copy link
Contributor

rultor commented Mar 31, 2015

@rultor merge please

@longtimeago Thanks for your request. @yegor256 Please confirm this.

@yegor256
Copy link
Member

@rultor merge this

@longtimeago
Copy link

@rultor merge?

@rultor
Copy link
Contributor

rultor commented Apr 1, 2015

@rultor merge this

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

@rultor rultor merged commit fe9f52d into jcabi:master Apr 1, 2015
@rultor
Copy link
Contributor

rultor commented Apr 1, 2015

@rultor merge this

@yegor256 Done! FYI, the full log is here (took me 5min)

@rultor
Copy link
Contributor

rultor commented Apr 1, 2015

@rultor merge?

@longtimeago Thanks for your request. @yegor256 Please confirm this.

@dmarkov
Copy link

dmarkov commented Apr 3, 2015

@longtimeago Thanks a lot, I just topped your account for 17 mins, transaction ID 54681633, total time was 49 hours and 9 mins. you're getting extra minutes here (c=2). +17 added to your rating, at the moment it is: +3723

@dmarkov
Copy link

dmarkov commented Apr 3, 2015

@rultor deploy now pls

@rultor
Copy link
Contributor

rultor commented Apr 3, 2015

@rultor deploy now pls

@dmarkov OK, I'll try to deploy now. You can check the progress here

@rultor
Copy link
Contributor

rultor commented Apr 3, 2015

@rultor deploy now pls

@dmarkov Done! FYI, the full log is here (took me 5min)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants