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

#25 Implemented SizeDecor class #28

Merged
merged 4 commits into from
Aug 19, 2014
Merged

#25 Implemented SizeDecor class #28

merged 4 commits into from
Aug 19, 2014

Conversation

carlosmiranda
Copy link
Contributor

No description provided.

@dmarkov
Copy link

dmarkov commented Aug 19, 2014

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

@dmarkov
Copy link

dmarkov commented Aug 19, 2014

@yegor256 please review this PR

@yegor256
Copy link
Member

@rultor pls merge

@rultor
Copy link
Contributor

rultor commented Aug 19, 2014

@rultor pls merge

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

@rultor
Copy link
Contributor

rultor commented Aug 19, 2014

@rultor pls merge

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

@carlosmiranda
Copy link
Contributor Author

@yegor256 it should be fixed now, please try again

@yegor256
Copy link
Member

@rultor try to merge again

@rultor
Copy link
Contributor

rultor commented Aug 19, 2014

@rultor try to merge again

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

@rultor rultor merged commit aca542d into jcabi:master Aug 19, 2014
@rultor
Copy link
Contributor

rultor commented Aug 19, 2014

@rultor try to merge again

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

@dmarkov
Copy link

dmarkov commented Aug 21, 2014

@yegor256 I added 16 mins to your account, many thanks for working with the project!

@dmarkov
Copy link

dmarkov commented Aug 21, 2014

@rultor please deploy

@rultor
Copy link
Contributor

rultor commented Aug 21, 2014

@rultor please deploy

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

@rultor
Copy link
Contributor

rultor commented Aug 21, 2014

@rultor please deploy

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

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.

None yet

4 participants