Skip to content

Conversation

@jowg-amazon
Copy link

  • add tests and fix error msg

  • fix feeds store

  • fix msg

  • add return so flushIOCs isn't called on failure


Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

* add tests and fix error msg

Signed-off-by: Joanne Wang <[email protected]>

* fix feeds store

Signed-off-by: Joanne Wang <[email protected]>

* fix msg

Signed-off-by: Joanne Wang <[email protected]>

* add return so flushIOCs isn't called on failure

Signed-off-by: Joanne Wang <[email protected]>

---------

Signed-off-by: Joanne Wang <[email protected]>
@jowg-amazon jowg-amazon requested a review from AWSHurneyt as a code owner July 17, 2024 21:03
@AWSHurneyt AWSHurneyt self-requested a review July 17, 2024 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants