dynalite icon indicating copy to clipboard operation
dynalite copied to clipboard

Add TransactWriteItems

Open ryanblock opened this issue 1 year ago • 3 comments

Thank you for helping out! ✨

We really appreciate your commitment to improving Dynalite

To maintain a high standard of quality in our releases, before merging every pull request we ask that you've completed the following:

  • [ ] Forked the repo and created your branch from main
  • [ ] Made sure tests pass (run npm it from the repo root)
  • [ ] Expanded test coverage related to your changes:
    • [ ] Added and/or updated unit tests (if appropriate)
    • [ ] Added and/or updated integration tests (if appropriate)
  • [ ] Updated relevant documentation
  • [ ] Summarized your changes in changelog.md
  • [ ] Linked to any related issues, PRs, etc. below that may relate to, consume, or necessitate these changes

Please also be sure to completed the CLA (if you haven't already).

Learn more about contributing to Architect here.

Thanks again!

ryanblock avatar Sep 03 '23 21:09 ryanblock

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 4 committers have signed the CLA.

:white_check_mark: ryanblock
:x: n0286293
:x: tume
:x: jpeterschmidt


n0286293 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Sep 03 '23 21:09 CLAassistant

@ryanblock What is the must fix here to get this merged? If we are just a few hours of work away I'd be happy pitch in.

jpshack-at-palomar avatar Feb 24 '24 10:02 jpshack-at-palomar

@jpshack-at-palomar please read my comments above; this PR has fundamental issues with how continuation passing works, and it cannot be merged in its current state. As a programmer, I do not feel confident making claims that anything is a few hours of work away, let alone something as complex as this.

I was hoping the author would help us get this PR into shape, but since we haven't heard back my plan has been to hopefully snag a few bits here and there, but probably rewrite. As ever, I'd be happy for someone in the community to start work on a fresh PR.

ryanblock avatar Feb 24 '24 16:02 ryanblock