correct IN usage in docstring by smtheard · Pull Request #101 · NerdWalletOSS/dynamorm · GitHub
Skip to content
This repository was archived by the owner on Apr 6, 2026. It is now read-only.

correct IN usage in docstring#101

Open
smtheard wants to merge 1 commit into
NerdWalletOSS:masterfrom
smtheard:patch-1
Open

correct IN usage in docstring#101
smtheard wants to merge 1 commit into
NerdWalletOSS:masterfrom
smtheard:patch-1

Conversation

@smtheard

Copy link
Copy Markdown

This is to fix an issue I found within the docs. __in is not a valid dynamoDB operator, it has to be __is_in in order to work both for dynamoDB and for dyanmORM.

@smtheard

Copy link
Copy Markdown
Author

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #101 (d79ea82) into master (418a0ed) will increase coverage by 0.19%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
+ Coverage   96.62%   96.81%   +0.19%     
==========================================
  Files          11       11              
  Lines        1007     1005       -2     
==========================================
  Hits          973      973              
+ Misses         34       32       -2     
Impacted Files Coverage Δ
dynamorm/model.py 99.45% <ø> (ø)
dynamorm/table.py 96.19% <0.00%> (+0.38%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 418a0ed...d79ea82. Read the comment docs.

@codecov-commenter

Copy link
Copy Markdown

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants