Skip to content

Don't force passing references to Client::query#99

Merged
msrd0 merged 2 commits into
mainfrom
query-as-ref
Oct 14, 2021
Merged

Don't force passing references to Client::query#99
msrd0 merged 2 commits into
mainfrom
query-as-ref

Conversation

@msrd0
Copy link
Copy Markdown
Collaborator

@msrd0 msrd0 commented Oct 4, 2021

Description

Don't force passing references to Client::query.
Passing references is still supported though as I've added a wildcard impl:
impl<Q: Query> Query for &Q

Checklist

  • Formatted code using cargo fmt --all
  • Linted code using clippy cargo clippy --all-targets --all-features -- -D warnings
  • Updated README.md using cargo readme -r influxdb -t ../README.tpl > README.md
  • Reviewed the diff. Did you leave any print statements or unnecessary comments?
  • Any unfinished work that warrants a separate issue captured in an issue with a TODO code comment

This is still supported though as I've added a wildcard impl:
`impl<Q: Query> Query for Q`
@msrd0 msrd0 merged commit 988d354 into main Oct 14, 2021
@msrd0 msrd0 deleted the query-as-ref branch October 14, 2021 20:43
@msrd0 msrd0 added this to the 0.5.0 milestone Oct 31, 2021
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.

1 participant