feat: allow skipping cf contents in "db-utils copy" (#420) #213
holtgreweAnnotations
3 warnings
|
release-please
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: GoogleCloudPlatform/release-please-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
writing `&Vec` instead of `&[_]` involves a new object where a slice will do:
src/genes/cli/data.rs#L372
warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do
--> src/genes/cli/data.rs:372:28
|
372 | fn serialize_vec<S>(x: &Vec<String>, s: S) -> Result<S::Ok, S::Error>
| ^^^^^^^^^^^^ help: change this to: `&[String]`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
= note: `#[warn(clippy::ptr_arg)]` on by default
|
|
this expression creates a reference which is immediately dereferenced by the compiler:
src/db_utils/cli/copy.rs#L202
warning: this expression creates a reference which is immediately dereferenced by the compiler
--> src/db_utils/cli/copy.rs:202:35
|
202 | if args.skip_cfs.contains(&cf_name) {
| ^^^^^^^^ help: change this to: `cf_name`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
= note: `#[warn(clippy::needless_borrow)]` on by default
|