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

DyBase.java:49-50: Add dynamo maven plugin and... #321

Closed
davvd opened this issue Sep 24, 2014 · 14 comments
Closed

DyBase.java:49-50: Add dynamo maven plugin and... #321

davvd opened this issue Sep 24, 2014 · 14 comments

Comments

@davvd
Copy link
Collaborator

davvd commented Sep 24, 2014

Puzzle 290-5f41f67f in src/main/java/com/thindeck/dynamo/DyBase.java:49-50 has to be resolved: Add dynamo maven plugin and create IT tests for dynamo based classes.

@davvd
Copy link
Collaborator Author

davvd commented Sep 24, 2014

@longtimeago please help, the task is yours

longtimeago pushed a commit to longtimeago/thindeck that referenced this issue Sep 28, 2014

Verified

This commit was signed with the committer’s verified signature. The key has expired.
suzuki-shunsuke Shunsuke Suzuki
longtimeago pushed a commit to longtimeago/thindeck that referenced this issue Sep 30, 2014

Verified

This commit was signed with the committer’s verified signature. The key has expired.
suzuki-shunsuke Shunsuke Suzuki
longtimeago pushed a commit to longtimeago/thindeck that referenced this issue Oct 1, 2014

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
longtimeago pushed a commit to longtimeago/thindeck that referenced this issue Oct 1, 2014

Verified

This commit was signed with the committer’s verified signature. The key has expired.
suzuki-shunsuke Shunsuke Suzuki
@longtimeago
Copy link
Contributor

@davvd please, close the issue. It's merged to default

@longtimeago
Copy link
Contributor

@yegor256 Please, close the issue. Thank you

@yegor256
Copy link
Owner

yegor256 commented Oct 5, 2014

@longtimeago did you remove the original puzzle?

longtimeago pushed a commit to longtimeago/thindeck that referenced this issue Oct 5, 2014

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@longtimeago
Copy link
Contributor

@yegor256 looks like no, sorry. Here is PR with todo removed #340

@davvd
Copy link
Collaborator Author

davvd commented Oct 6, 2014

@longtimeago this task is in your hands for 11 days already

@longtimeago
Copy link
Contributor

@davvd waiting for review #340

@yegor256
Copy link
Owner

yegor256 commented Oct 6, 2014

@davvd a bit more time required here

@davvd
Copy link
Collaborator Author

davvd commented Oct 6, 2014

@davvd a bit more time required here

@yegor256 of course, take your time

@longtimeago
Copy link
Contributor

@yegor256 #340 Merged to master

@davvd
Copy link
Collaborator Author

davvd commented Oct 8, 2014

Closed, thanks all

@davvd davvd closed this as completed Oct 8, 2014
@davvd
Copy link
Collaborator Author

davvd commented Oct 8, 2014

@longtimeago Thanks a lot, I just topped your account for 30 mins, transaction ID 45778199

@davvd
Copy link
Collaborator Author

davvd commented Nov 10, 2014

these puzzles are in progress: 321-7e1601a9/#341, 341-7e1601a9/#413, 341-c00e1503/#419 (once finished this ticket is done)

@davvd
Copy link
Collaborator Author

davvd commented Dec 24, 2014

last puzzled solved 341-c00e1503/#419

0pdd referenced this issue Jan 29, 2018

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants