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

#23 fixed documentation in section Maintaining SQL Transactions Through JDBC #32

Merged
merged 1 commit into from
Jan 19, 2015
Merged

Conversation

asaen
Copy link

@asaen asaen commented Dec 27, 2014

No description provided.

@dmarkov
Copy link

dmarkov commented Dec 31, 2014

Let me find a reviewer for this pull request, thanks for submitting it

@dmarkov
Copy link

dmarkov commented Dec 31, 2014

@longtimeago please review

@longtimeago
Copy link

@rultor please, merge

@rultor
Copy link
Contributor

rultor commented Dec 31, 2014

@rultor please, merge

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

@rultor
Copy link
Contributor

rultor commented Dec 31, 2014

@rultor please, merge

@longtimeago Oops, I failed. You can see the full log here (spent 4min)

@longtimeago
Copy link

have to wait for #33

@asaen
Copy link
Author

asaen commented Jan 18, 2015

@longtimeago #33 seems to be closed now, could you please check again?

@longtimeago
Copy link

@rultor please, merge

@rultor
Copy link
Contributor

rultor commented Jan 19, 2015

@rultor please, merge

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

@yegor256
Copy link
Member

@rultor merge it

@rultor
Copy link
Contributor

rultor commented Jan 19, 2015

@rultor merge it

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

@rultor rultor merged commit 2ad5ab0 into jcabi:master Jan 19, 2015
@rultor
Copy link
Contributor

rultor commented Jan 19, 2015

@rultor merge it

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

@dmarkov
Copy link

dmarkov commented Jan 21, 2015

@longtimeago 22 mins was added to your account, many thanks for your contribution!; +22 to your rating, your total score is +1199

@dmarkov
Copy link

dmarkov commented Jan 21, 2015

@rultor deploy pls

@rultor
Copy link
Contributor

rultor commented Jan 21, 2015

@rultor deploy pls

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

@rultor
Copy link
Contributor

rultor commented Jan 21, 2015

@rultor deploy pls

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

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