Skip to content

Conversation

@msrd0
Copy link
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.

2 participants